Skip to content

fix(home): stack hero content on mobile - #115

Merged
nibzard merged 1 commit into
mainfrom
agent/fix-mobile-home-hero
Aug 3, 2026
Merged

fix(home): stack hero content on mobile#115
nibzard merged 1 commit into
mainfrom
agent/fix-mobile-home-hero

Conversation

@nibzard

@nibzard nibzard commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

What changed

Why

The longer canonical identity sentence introduced in #113 made the fixed 188px logo and copy compete
for the same narrow mobile row. Because the row was bottom-aligned, the logo was pushed down while
the text wrapped into a very narrow column.

Impact

The homepage hero is readable at phone widths, the logo keeps a stable position, and the
answer-engine identity copy remains visible and unchanged.

Validation

  • bunx biome check 'app/(home)/_pages/page.en.tsx'
  • bun run typecheck
  • rendered locally at 320px, 390px, 768px, and 1280px with no horizontal overflow

bun run check also ran, but the repository baseline currently reports missing trailing newlines
in scripts/seo/pulse/snapshots/tech-2026-07.json and
scripts/seo/pulse/snapshots/volumes-2026-07.json; this PR does not touch those files.

@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 3, 2026 9:22am

Request Review

@nibzard
nibzard marked this pull request as ready for review August 3, 2026 09:47
@nibzard
nibzard merged commit 9258122 into main Aug 3, 2026
2 of 3 checks passed
@nibzard
nibzard deleted the agent/fix-mobile-home-hero branch August 3, 2026 09:47
nibzard added a commit that referenced this pull request Aug 4, 2026
The assertion matched the whole class attribute as one string, so #115
adding `self-center sm:self-auto` for mobile stacking broke it even though
the invariant it guards still held. The logo must keep `shrink-0` so the
hero's flex row cannot squash it; responsive alignment classes around it are
free to change.

Verified the assertion still fails when shrink-0 is actually removed.
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