PR Custody

fix(toolkits): remove .skin-landing shim + unused trial fonts (fixes navbar dropdown font)

@Malayvasachecks n/achecks…claude/recursing-black-7c58b0 → master105 files · +27 −231updated 5h ago

▸Description · 1 noise

What & why

Started as a fix for the navbar dropdown rendering a different font on /toolkits, and grew into removing the root cause entirely.

The bug

The /toolkits listing page wrapped its content in <main className="skin-landing">. That class defined a universal !important font override (.skin-landing * { font-family: ABC Diatype }) plus a pile of attribute-selector overrides meant to re-skin the original odysseus-ported markup. Because the rule targets every element directly, it broke font inheritance for unclassed dropdown/footer spans that inherit font-mono — flipping them to ABC Diatype only under /toolkits.

The realization

The listing-page components (ToolkitsMarqueeHero, ToolkitBrowser, ToolkitsTestimonials) are already built with our design tokens and emit none of the inline rgb(...) / Roobert / Flecha styles the shim was written to override. So .skin-landing was vestigial — its surface/CTA/border rules matched nothing, and its only live effect was the broad font rules that leaked into the global navbar and footer.

Changes

Delete the .skin-landing block from globals.css and drop the class from the listing page's <main>.
Delete the trial-font @font-face declarations (Flecha Test S, Roobert TRIAL, Roobert Mono TRIAL) — unused anywhere — and remove the orphaned .otf files under public/toolkits/.
Replace remaining hardcoded 'Roobert …' inline styles in Breadcrumb.tsx and RubeInstallCodeBlock.tsx with var(--font-sans) / var(--font-mono). These render on the detail/framework pages, which never had the shim — so they were rendering off-brand Roobert today.
Repoint the dead TYPOGRAPHY.fontHeading/fontBody tokens off the trial fonts.
Navbar + Footer were also lifted out of the wrapper (earlier commits) — kept that way since it's correct HTML semantics regardless.

Net: zero references to Roobert / Flecha / Mono TRIAL / skin-landing remain in the codebase.

Verification

✅ pnpm typecheck passes
✅ grep confirms no remaining refs to the removed fonts/class
⚠️ Needs a visual QA pass on staging. The /toolkits listing page can't be exercised in the worktree (Strapi data API unreachable → error fallback), so the hero/browser/testimonials should be eyeballed on a working preview to confirm they render identically without the shim. They're token-based, so no change is expected.

🤖 Generated with Claude Code

▸ 1 bot/status comment hidden

@vercel[bot]5h ago

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
landing-new	Ready	Preview, Comment	Jun 13, 2026 5:22am

loading diff…