Skip to content

Fix portrait alignment, mobile menu behavior, and hero spacing - #30

Open
hoektoe wants to merge 3 commits into
new-site-redesignfrom
codex/fix-mobile-hero-navigation
Open

Fix portrait alignment, mobile menu behavior, and hero spacing#30
hoektoe wants to merge 3 commits into
new-site-redesignfrom
codex/fix-mobile-hero-navigation

Conversation

@hoektoe

@hoektoe hoektoe commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep Michelle’s face and full head in frame in the shared mobile consultation CTA
  • right-align the About hero portrait at both mobile and desktop widths while preserving the default left alignment for other hero instances
  • round both top corners on every shared background-hero image, including About, Therapy Services, and the individual service pages
  • close the mobile navigation dialog and reset the page to the top when a menu link is selected
  • remove the empty hero-footer gap and mobile-only bottom padding that created excessive whitespace below shared hero sections

Root cause

The CTA used the default centered object-cover crop for a portrait image inside a square mobile frame. The About hero had no explicit media alignment option, and its desktop media column did not consume the available flexible width. Shared hero rounding was applied only to the overflow frame, so the image corner away from the frame edge remained square. The mobile menu links did not explicitly close their dialog or handle same-route scroll position. The shared hero also rendered an empty footer container, so its gap-16 combined with hero and following-section padding.

Validation

  • Prettier check on changed files
  • focused ESLint on the changed shared components
  • RESEND_API_KEY=re_dummy yarn build
  • Playwright at 390×844 and 1440×1000: About portrait is right-aligned with zero frame-edge delta, other hero instances retain their default left alignment, and no console errors or framework overlays were detected
  • Playwright on About and Therapy Services at 390×844 and 1440×1000: both image top corners compute to an 8px radius
  • Playwright at 390×844: consultation portrait uses the corrected 50% 20% focal point; About hero-to-heading gap is 64px; same-route mobile menu selection closes the dialog and returns scrollY to 0

Existing lint debt

A full-file ESLint run on src/app/about/page.tsx still reports six pre-existing react/no-unescaped-entities errors in page copy; this PR does not alter those lines.

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
michellesmit Ready Ready Preview, Comment Jul 21, 2026 8:21am

@hoektoe
hoektoe marked this pull request as ready for review July 21, 2026 07:28
@hoektoe hoektoe changed the title Fix mobile portrait framing, menu behavior, and hero spacing Fix portrait alignment, mobile menu behavior, and hero spacing Jul 21, 2026
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