fix: canonicalize math routing and wire orphan pages into sidebar navigation#56
Closed
fix: canonicalize math routing and wire orphan pages into sidebar navigation#56
Conversation
…vigation Navigation IA cleanup covering math route consolidation and sidebar discoverability improvements across multiple sections. Math routing: - Add /math hub page (MathLandingView) with cards linking to Refresh and Bridge - Add canonical child routes /math/refresh and /math/bridge - Legacy /math-refresh and /math-bridge redirect to canonical paths - Reference deep links untouched Sidebar wiring (Stage A1 - Learning + Career): - Add Cognitive Toolkit and Behavioral Design to Learning section - Add Career Foundations to Career section - Extend Learning header active state for new items + broadened math_ Sidebar wiring (Stage A2 - Projects + Business): - Add OSS Projects and HF Projects to Projects section - Add Brand Studio and Execution Playbook to Business section - Extend parent header active states for new items Section rename (Stage B1): - Rename "Side Projects" to "Business" (label + aria only) Docs deep-dives (Stage B2): - Add 6 deep-dive sub-items under Docs > How It Works: Ingestion Pipeline, Chat Pipeline, Career Intelligence, Observability, Architecture Reference, MCP Ecosystem - Extend Docs header active state for all deep-dive routes - Add mcp_ecosystem ViewId and pathname detection for /how-mcp-works - /how-gpu-works intentionally left in Reference section Verified: tsc 0 errors, lint 0 errors, Playwright confirmed navigation and active state for all new items across all affected routes.
Owner
Author
|
Superseded by two smaller scoped PRs:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/mathhub page with canonical child routes/math/refreshand/math/bridge, with legacy redirects from/math-refreshand/math-bridgeChanges by stage
Math routing canonicalization
MathLandingViewwith two entry cards (Refresh + Bridge)/math/refreshand/math/bridgeas canonical routes/math-refreshand/math-bridgeredirect with search params preserved/reference?section=math-refresh) untouchedStage A1 - Learning + Career wiring
Stage A2 - Projects + Business wiring
Stage B1 - Section rename
Stage B2 - Docs deep-dives
/how-gpu-worksintentionally left in Reference > GPUs (different domain)/how-mcp-worksneeded newmcp_ecosystemViewId + pathname detection (was the only true orphan with zero sidebar infrastructure)Test plan
npx tsc --noEmit- 0 errorspnpm lint- 0 errors (11 pre-existing warnings)data-active="true"strictly verified on all 6 deep-dive routes/how-gpu-worksconfirmed untouched (remains in Reference only)🤖 Generated with Claude Code