Skip to content

Add category menu row to the header#11

Merged
700799 merged 1 commit into
mainfrom
claude/header-category-rows
May 30, 2026
Merged

Add category menu row to the header#11
700799 merged 1 commit into
mainfrom
claude/header-category-rows

Conversation

@700799

@700799 700799 commented May 30, 2026

Copy link
Copy Markdown
Owner

Add a category menu row to the header

Per the request for a second/third row of category menus as part of the header, this adds a second header row that lists the theme categories as quick links.

Changes

  • Header.tsx — a new "Categories" row beneath the main nav, listing every theme category (Power & Politics, Identity & Belonging, Morality & Justice, Love & Family, Race & Class, Dreams & Ambition, Freedom & Society, Science & Mortality). On narrow screens it scrolls horizontally. The list is sourced from allThemes() so it stays in sync with the filter bar.
  • Each category links to /?theme=<Category>#browse.
  • BrowseSection.tsx — reads the ?theme= query param on mount (client-side) and pre-selects that theme, so clicking a category lands you on a filtered grid at the browse section. Invalid params are ignored.
  • books/[slug]/page.tsx — bumped the sticky in-page sub-nav offset (top-12top-[76px]) so it clears the now two-row header.

Verify

  • npm run build succeeds (50 routes export); the PR build CI job runs it too.
  • Confirmed the "Categories" row and correctly URL-encoded ?theme= links render in the header on both the home page and guide pages.

https://claude.ai/code/session_01Egnnw9v4T3ppsAQveFnijo


Generated by Claude Code

Add a second header row listing the theme categories (Power & Politics,
Identity & Belonging, etc.) as quick links. Each links to
/?theme=<Category>#browse, and BrowseSection now reads the ?theme= query
param on mount to land on a pre-filtered grid.

Bump the guide page's sticky sub-nav offset (top-12 -> top-[76px]) to
account for the now two-row header.
@700799 700799 marked this pull request as ready for review May 30, 2026 18:20
@700799 700799 merged commit d040bc6 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