Restore teal accent, flatten sidebar background, drop body letter-spacing - #273
Open
dcrawbuck wants to merge 1 commit into
Open
Restore teal accent, flatten sidebar background, drop body letter-spacing#273dcrawbuck wants to merge 1 commit into
dcrawbuck wants to merge 1 commit into
Conversation
- Accent tokens (`primary`, `ring`) move from orange back to teal. Dark mode uses teal-300; light mode uses teal-400, since teal's bright step is a light cyan that fails contrast on the white canvas. - The sidebar aside paints `bg-fd-card`, which sat a depth tier above the page and read as a differently-colored panel. Pin card to the canvas inside `#nd-sidebar` so the hairline border alone separates them. - Drop the -0.01em body letter-spacing; body now uses Manrope's natural tracking. Heading tracking is unchanged. `--color-fd-ring` is the sole `focus-visible:ring-2` indicator across the app, so the light-mode value is contrast-bound: teal-300 measures 1.76:1 on white (under the 3:1 non-text minimum), teal-400 measures 6.75:1.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
superwall-docs | 5c8e6b1 | Commit Preview URL Branch Preview URL |
Aug 19 2026, 05:48 PM |
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
1 participant
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.
Three theme changes in
src/styles/app.css: theprimary/ringaccent tokens move from orange back to teal, the sidebar background now matches the page canvas, and the-0.01embody letter-spacing is removed so body text uses Manrope's natural tracking (heading tracking is unchanged).The sidebar aside paints
bg-fd-card, which sat a depth tier above the canvas and read as a differently-colored panel, so--color-fd-cardis pinned to the background inside#nd-sidebar— scoped there so cards and popovers elsewhere are unaffected, and the hairline border alone separates nav from content.Dark mode uses
teal-300, but light mode usesteal-400for both accent text and the focus ring:--color-fd-ringis the solefocus-visible:ring-2indicator across the app, and teal's bright step is a light cyan that measures only 1.76:1 on the white canvas, under the 3:1 minimum for a non-text indicator, whereteal-400measures 6.75:1.Verified with
bun run build:cfandbun test(64 pass, 0 fail).Note
Low Risk
CSS-only visual and token changes with no logic, auth, or data impact; main risk is contrast or focus-ring visibility regressions in light mode.
Overview
Docs theme in
app.cssmoves the Fumadocsprimary/ringaccent from orange back to teal (links, active nav/TOC, prose links, focus rings, Ask AI shimmer comments). Light mode usesteal-400for both accent text and the focus ring so rings meet 3:1 on the white canvas; dark mode andprefers-color-schemefallbacks useteal-300.The sidebar no longer reads as a separate panel: inside
#nd-sidebar,--color-fd-cardis pinned to--color-fd-backgroundso nav shares the page canvas and only the hairline border separates it.Body text drops the global
letter-spacing: -0.01emso Manrope uses its natural tracking; heading tracking is unchanged.Reviewed by Cursor Bugbot for commit 5c8e6b1. Bugbot is set up for automated code reviews on this repo. Configure here.