Skip to content

Top-bar one-word quick-nav; remove bottom-right floating menu#8

Merged
700799 merged 2 commits into
mainfrom
claude/top-bar-nav
May 30, 2026
Merged

Top-bar one-word quick-nav; remove bottom-right floating menu#8
700799 merged 2 commits into
mainfrom
claude/top-bar-nav

Conversation

@700799

@700799 700799 commented May 30, 2026

Copy link
Copy Markdown
Owner

Top-bar quick-nav, floating menu removed

Per the request to make navigation a top bar with one-word links for quick browsing, and to remove the bottom-right floating button.

Changes

  • Header.tsx — the top bar now shows an always-visible, one-word quick-nav on every screen size (previously the links were hidden on mobile, which is why the floating menu existed):
    Home · Browse · Highlights · Rewards
    The level/XP chip (HeaderStats) stays on the right. The "Deep reading guides" tagline is hidden on small screens so the row fits comfortably on mobile.
  • layout.tsx — removed the <FloatingMenu /> render and import.
  • FloatingMenu.tsx — deleted (only layout.tsx referenced it).

Note

The floating menu's "back to top" affordance goes away with it, as expected; guide pages keep their own sticky in-page quick-nav.

Verify

  • npm run build succeeds (42 routes export); the PR build CI job runs it too.
  • Confirmed the four nav words render in the header and there is no floating-button markup (aria-label="Open menu") anywhere in out/.

https://claude.ai/code/session_01Egnnw9v4T3ppsAQveFnijo


Generated by Claude Code

claude added 2 commits May 30, 2026 15:32
- Header now shows always-visible one-word quick-nav (Home, Browse,
  Highlights, Rewards) on every screen size, replacing the sm-only links
- Delete the bottom-right FloatingMenu component and its layout render
- Hide the brand tagline on small screens so the nav row fits on mobile
@700799 700799 marked this pull request as ready for review May 30, 2026 15:43
@700799 700799 merged commit 9ad5f52 into main May 30, 2026
1 check 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.

2 participants