Skip to content

feat(viewers): full-screen reading for document, image & map viewers - #381

Merged
slaveofcode merged 2 commits into
developfrom
feat/viewers-fullscreen
Sep 11, 2026
Merged

feat(viewers): full-screen reading for document, image & map viewers#381
slaveofcode merged 2 commits into
developfrom
feat/viewers-fullscreen

Conversation

@slaveofcode

Copy link
Copy Markdown
Owner

What

Adds a Full screen toggle to the viewer tools (option C of the mobile-viewer work), so any of them can be read full-screen — great on a phone.

  • New shared ExpandableViewer wrapper, built on the existing useExpand hook (native Fullscreen API + iOS CSS-overlay fallback, Esc to exit) — the same mechanism the games/Teleprompter use.
  • Wrapped 13 viewers: PPTX, Legacy .doc, GEDCOM, EML, DICOM, iWork, DOCX, Spreadsheet/CSV, EPUB, ODT, Image, SVG, GeoJSON/GPX/KML.

Tests

  • E2E e2e/tools/viewer-fullscreen.spec.ts: toggle → Exit → toggle on svg-viewer + image-viewer.
  • Registry render smoke covers all 13 wrapped pages (verified green locally).
  • Lint 0 errors; full build green (all viewer pages build). No SEO changes — existing tools, feature-only enhancement.

Follows PR #379 (markdown viewer). Next: per-tool PWA install gets its own spec.

Promote to production: Markdown viewer — open .md files + full-screen
Add a shared ExpandableViewer wrapper (built on the useExpand hook:
native Fullscreen API + iOS CSS-overlay fallback, Esc to exit) and wrap
13 viewer tools with it so each can be read full-screen — handy on a
phone: PPTX, Legacy .doc, GEDCOM, EML, DICOM, iWork, DOCX, Spreadsheet/
CSV, EPUB, ODT, Image, SVG and GeoJSON/GPX/KML viewers.

Happy-path E2E asserts the toggle on two representative viewers; the
registry render smoke covers all 13.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
🔵 In progress
View logs
goodwebtools 8bed496 Sep 11 2026, 09:02 AM

@slaveofcode
slaveofcode merged commit 9d6ea92 into develop Sep 11, 2026
2 of 3 checks passed
@slaveofcode
slaveofcode deleted the feat/viewers-fullscreen branch September 11, 2026 09:08
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.

1 participant