Skip to content

fix(ui): wrap member section nav 2-up on mobile (#1331)#1335

Merged
jonfroehlich merged 1 commit into
masterfrom
1331-mobile-section-nav-2up
Jun 18, 2026
Merged

fix(ui): wrap member section nav 2-up on mobile (#1331)#1335
jonfroehlich merged 1 commit into
masterfrom
1331-mobile-section-nav-2up

Conversation

@jonfroehlich

Copy link
Copy Markdown
Member

Summary

Fixes #1331. On phones, the sticky member-section-nav-list (Projects / Papers / Videos / Talks) wrapped unevenly, orphaning Talks onto its own line.

This pins each link to ~half-row width below the 576px phone breakpoint, so the four sections wrap a clean 2-up grid:

Projects (8/8)    Papers (6/106)
Videos (6/14)     Talks (8/95)

Why this approach

  • vs. forcing one line: at ~320px that needs tiny text or horizontal scroll and shrinks tap targets. The 2-up layout keeps links comfortably tappable (consistent with the WCAG 2.5.5 touch-target care elsewhere in this file).
  • vs. CSS Grid: reusing the list's existing flex-wrap + justify-content: center means a member with only three sections gets the lone last link centered on row 2 instead of left-orphaned.
  • 576px matches this file's existing "phone" breakpoint (the mobile grid caps use the same).

Changes

  • website/static/website/css/member.css: one @media (max-width: 576px) block (CSS only — no template/JS change).

Testing

CSS-only layout change; no automated tests apply. Verified the wrap math (2 × calc(50% - gap) + gap = 100% - gap, no overflow). No before/after screenshot per maintainer request.

🤖 Generated with Claude Code

The sticky section nav (Projects / Papers / Videos / Talks) wrapped
unevenly on phones, orphaning "Talks" onto its own line. Pin each link
to ~half-row width below 576px so the four sections wrap a clean 2-up
grid. Reusing the existing flex-wrap + justify-content:center also
centers a lone last item for members with only three sections.

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

Copy link
Copy Markdown
Member Author

Looks like this now!

image

@jonfroehlich jonfroehlich merged commit 9087960 into master Jun 18, 2026
2 checks passed
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.

On mobile, the new sticky header member-section-nav-list splits awkwardly into two lines

1 participant