Skip to content

journal: archive wide-viewport whitespace investigation (resolved by #76)#77

Merged
wilsonwangdev merged 1 commit into
mainfrom
journal/wide-viewport-whitespace
Jun 1, 2026
Merged

journal: archive wide-viewport whitespace investigation (resolved by #76)#77
wilsonwangdev merged 1 commit into
mainfrom
journal/wide-viewport-whitespace

Conversation

@wilsonwangdev

Copy link
Copy Markdown
Owner

Summary

Archives a 2026-05-08 investigation that had been sitting uncommitted in a feature worktree until today, and closes the loop with the fix that just shipped.

journal/2026-05-08-wide-viewport-whitespace.md records:

  • What happened — capturing 1280×720 social-launch screenshots revealed ~35% horizontal whitespace because the site had exactly one breakpoint (max-width: 640px) and a fixed --max-w: 52rem
  • Root cause — review processes (Lighthouse, frontend-interaction-review skill) optimized for correctness, not visual density at wide viewports
  • Resolution — closed by build: align homepage spacing and section order with design intent #76, which added @media (min-width: 1100px) widening --max-w to 64rem. The two alternative options (2-col layout, decorative frame) were rejected because they conflict with rules/design-intent.md §5 (wide viewports widen content, not whitespace or chrome)
  • Prevention — ROADMAP item to add a viewport-density check to the frontend-interaction-review skill's next revision

ROADMAP.md gains three matching [x] rows in Now for #74 / #75 / #76.

.gitignore adds assets/social/ — launch screenshots are local promotional artifacts. The first batch (May 8) is staying on disk but won't be committed.

Why this matters

The journal entry is exactly the "failures captured for agent self-evolution" use case in CLAUDE.md. Without committing it, the institutional memory of why PR #76 added that specific breakpoint at 1100px (and not, say, 1280px or 1440px) would have been lost. This PR makes the chain 1280px screenshot reveals issue → root cause → option list → chosen fix → rule §5 discoverable from a single file.

Tested

Lint-only / docs-only changes. No build impact.

#76

The 2026-05-08 social-launch screenshot batch surfaced that the homepage
showed ~35% horizontal whitespace at 1280-wide viewports because the
site had only one media query (max-width: 640px). The investigation
sat in a feature worktree alongside the launch assets and was never
committed at the time.

PR #76 has now resolved it: the new `min-width: 1100px` breakpoint
widens `--max-w` from 52rem to 64rem, applying option 1 from the
journal's option list. The two-column layout and decorative-frame
options were rejected because they conflict with rules/design-intent.md
§5 — wide viewports widen content, not whitespace, and not chrome.

Also:
- ROADMAP gets three matching [x] entries for PR #74/#75/#76 in Now
- .gitignore adds assets/social/ — launch screenshots are local-only
  promotional artifacts, not committed alongside the source

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agent-master-handbook Ready Ready Preview, Comment Jun 1, 2026 4:48pm

@wilsonwangdev wilsonwangdev merged commit 8fdafa6 into main Jun 1, 2026
3 checks passed
@wilsonwangdev wilsonwangdev deleted the journal/wide-viewport-whitespace branch June 1, 2026 16:49
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