Add /docs and /articles sections - #1
Conversation
Adds a documentation section (what Suede DNA is, sourcing and verification, search syntax, FAQ) and five archive-voice essays on rig history and tone genealogy. Wires both into header/footer nav, sitemap.ts, and llms.txt.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: aaf9659485
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| { href: '/docs', label: 'DOCS' }, | ||
| { href: '/articles', label: 'ARTICLES' }, |
There was a problem hiding this comment.
On narrow mobile viewports (for example, 375px), adding these two labels makes the fixed 64px header's minimum content width exceed the viewport: the logo, gap-5, five non-wrapping nav labels, four gap-6 gaps, and horizontal px-6 padding cannot shrink. The header therefore overflows horizontally and its navigation is clipped/off-screen; add a compact mobile treatment, reduce/hide links responsively, or make the navigation scrollable.
Useful? React with 👍 / 👎.
| backLabel?: string; | ||
| maxWidth?: string; | ||
| }) { | ||
| return ( |
There was a problem hiding this comment.
Every new detail route using this shell renders a <main> with no id="archive", while the root layout's only skip link targets #archive (src/app/layout.tsx:113). On these docs and article pages, keyboard users who activate “SKIP TO ARCHIVE” are left at the same position because the fragment target does not exist; give this main an appropriate target ID or make the global skip-link destination route-aware.
Useful? React with 👍 / 👎.
2 participants
Summary
/docssection: what Suede DNA is, sourcing and verification (honest about the guitargeek-archives → manifest pipeline, what is and isn't verified), search/filter syntax, and an FAQ with FAQPage JSON-LD./articlessection with five full essays (800-1500 words each) in the site's archival voice: Clapton's 1964-1967 signal chain, a history of the rig diagram as a documentary form, the 1980s-2000s virtuoso rig era, rig archaeology for ear-trained players, and the unphotographed parts of a signal chain (power, cables, mic placement).data/*.json,src/lib/search.ts,src/lib/filters.ts) — no invented sourcing process, no fabricated statistics.sitemap.ts, andpublic/llms.txt.PageShell/DocSectionmirror the About page's structure; no new hex literals, no new runtime dependencies).Test plan
pnpm typecheckpassespnpm lintpasses (no warnings)pnpm testpasses (80/80 vitest tests)pnpm buildpasses, all 11 new routes prerender as static contentpnpm startsmoke test: all new routes return 200, sitemap.xml includes them, header/footer nav renders, FAQPage JSON-LD present,/aboutunaffected