Skip to content

Fix report print stylesheet (#221)Fix report print stylesheet (#221) - #236

Merged
shossain786 merged 1 commit into
seleniumboot:masterfrom
AnubhavMaheshwari:fix/report-print-stylesheet
Nov 4, 2026
Merged

shossain786 merged 1 commit into
seleniumboot:masterfrom
AnubhavMaheshwari:fix/report-print-stylesheet

Conversation

@AnubhavMaheshwari

Copy link
Copy Markdown
Contributor

Summary
Addresses #221 by improving the report's print stylesheet.

Changes

  • Added print-specific styles to improve report readability.
  • Updated report printing behavior where necessary.
  • Improved the layout for printing and saving reports as PDF.

Testing

  • Command: .\mvnw.cmd test
  • Result: [Add the actual test result here once the run finishes.]

@shossain786

Copy link
Copy Markdown
Contributor

Thanks for picking this up, @AnubhavMaheshwari. The print CSS and the beforeprint/afterprint state handling look good. A few things before I can merge:

  • Line endings: report.js was converted from LF to CRLF, so the diff shows +454/−277 when the real change is about 177 added lines. Please restore LF (git config core.autocrlf input, then re-checkout and reapply your changes) so the diff is reviewable.
  • Test result: the PR description still has the "[Add the actual test result here]" placeholder. Please fill it in with the real result.
  • Command: .\mvnw.cmd test is Windows-only. ./mvnw test is what CI runs.
  • Manual check: please add a short note, or a screenshot of the PDF output, showing a report with a collapsed group and a failed test printed with its stack trace.

Happy to re-review as soon as these are in.

@AnubhavMaheshwari
AnubhavMaheshwari force-pushed the fix/report-print-stylesheet branch from b4aa21a to 4afe28a Compare November 3, 2026 05:46
@shossain786
shossain786 merged commit 61e9824 into seleniumboot:master Nov 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants