Skip to content

feat(report): rebrand HTML test report as Mobilewright - #6

Merged
gmegidish merged 2 commits into
mainfrom
fix/test-report-branding
Apr 2, 2026
Merged

gmegidish merged 2 commits into
mainfrom
fix/test-report-branding

Conversation

@krismuhi

Copy link
Copy Markdown
Member

Summary

  • Replaces all Playwright branding (title, favicon, header) with Mobilewright in HTML test reports
  • Adds a clickable Mobilewright logo in the report header that links to the GitHub repo
  • Screenshot thumbnails now render at max 250px width with click-to-fullscreen overlay (Escape or click to dismiss)

Changes

  • packages/mobilewright/src/reporter.ts — new brandReport() function that post-processes Playwright's HTML report output
  • packages/mobilewright/src/cli.ts — calls brandReport() after test run when --reporter html is used
  • packages/mobilewright/assets/mobilewright-icon.png — icon used as favicon and header logo
  • packages/mobilewright/package.json — includes assets/ in published files

Test plan

  • Run mobilewright test --reporter html and open the generated report
  • Verify browser tab shows "Mobilewright Test Report" with the mobilewright favicon
  • Verify header shows mobilewright logo + name, clicking it opens the GitHub repo
  • Verify screenshots render as 250px thumbnails
  • Click a screenshot and verify fullscreen overlay appears
  • Press Escape or click overlay to dismiss

- Replace Playwright title/favicon with Mobilewright branding
- Add header logo linking to the GitHub repo
- Constrain screenshot thumbnails to 250px with click-to-fullscreen overlay
- Post-process HTML report automatically when using --reporter html
@krismuhi

Copy link
Copy Markdown
Member Author

im not happy about this change.

@gmegidish
gmegidish merged commit b868fb2 into main Apr 2, 2026
1 check passed
@gmegidish
gmegidish deleted the fix/test-report-branding branch April 2, 2026 10:34
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