Codex Relay: zyra-project/zyra PR #113 — fix: overhaul print preview layout to fill full 3600px poster height

[Hackshaven]

Mirrored from zyra-project/zyra PR #113

Source: zyra-project/zyra#113

This PR is maintained by an automated relay. Changes should be made in the original downstream PR.

[Copilot]

Pull request overview

This PR performs a major architectural refactor of the poster HTML/CSS system, separating web and print layouts into distinct files. The web view (zyra-poster.html) remains scrollable and interactive, while a new dedicated print version (zyra-poster-print.html) provides a static 300 DPI layout at 14400×10800px (48×36 inches).

Note: There is a critical discrepancy - the PR description claims this is a relay of PR #113 about bumping netcdf4 dependency, but the actual changes are an extensive poster layout refactor with no dependency updates whatsoever.

Changes:

• Separated print layout into standalone _print.css (1740 lines) with fully self-contained styling at 300 DPI resolution
• Replaced interactive print mode toggle with static link to dedicated print HTML file
• Converted footer "Get Started" links from plain text divs to clickable badge anchors with icons
• Added new HTML fragments for print layout (_head-print.html, _body-open-print.html, _footer-print.html)
• Updated build script to generate both web and print versions, excluding gallery section from print
• Added screenshot automation scripts for print layout capture

Reviewed changes

Copilot reviewed 12 out of 14 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
poster/sections/_print.css	New 1740-line self-contained print stylesheet defining 14400×10800px grid layout at 300 DPI
poster/sections/_styles.css	Removed ~280 lines of body.poster-print CSS rules, replaced with comment redirecting to separate print file; updated footer badge styling from text-based links to interactive badges
poster/sections/sec-10-features.html	Converted 6 footer link divs to clickable anchor badges with inline SVG icons and structured label/value spans
poster/sections/_body-open.html	Replaced interactive print mode toggle with static link to zyra-poster-print.html
poster/sections/_footer.html	Removed ~60 lines of print mode toggle JavaScript; simplified to only handle responsive scaling and lightboxes
poster/sections/_head-print.html	New 9-line HTML head for print version with Google Fonts and inline CSS opening tag
poster/sections/_body-open-print.html	New 4-line body opening with poster div for print layout
poster/sections/_footer-print.html	New 3-line closing tags for print HTML
poster/scripts/build_poster.py	Enhanced to build two output files with separate fragment sets; excludes sec-09-gallery from print version
poster/scripts/screenshot.js	New 30-line Playwright script to capture 14400×10800px print layout screenshot
poster/scripts/screenshot-compare.js	New 72-line script for comparative screenshots but contains broken toggle-clicking logic
poster/html/zyra-poster.html	Generated output reflecting all source changes (removed print CSS, updated badges, simplified JS)
.gitignore	Added poster screenshot and preview artifact patterns

[Copilot]

The PR title mentions "fill full 3600px poster height," but the new print layout is actually 10800px high (14400×10800px at 300 DPI, which is 48×36 inches). The grid-template-rows sum to 10800px: 1200 + 2200 + 2600 + 3100 + 1700 = 10800. The 3600px mentioned in the title likely refers to the old 100 DPI layout (4800×3600px) that was removed from _styles.css. The title should be updated to reflect the actual 10800px height or clarify that this is a complete overhaul to 300 DPI.

[Copilot]

The new badge markup includes SVG icons without accessible text alternatives. While the badges have visible text labels (e.g., "GitHub", "PyPI"), screen readers may not properly interpret the SVG icons. Consider adding aria-label or title attributes to the SVG elements, or use aria-hidden="true" on the SVGs if they're purely decorative alongside the text.

[Copilot]

The PR title mentions "fix: overhaul print preview layout to fill full 3600px poster height," but the PR description indicates this is a relay of zyra-project/zyra PR #113, which is described as "chore(deps): bump netcdf4 from 1.7.1.post2 to 1.7.2" - a dependency update. However, the actual code changes in this PR are a major refactor of the poster HTML/CSS layout system, not a netcdf4 dependency bump. There's a significant discrepancy between the PR metadata and the actual changes.