fix(legal): use theme tokens so legal pages are readable - #15
Merged
Conversation
|
Someone is attempting to deploy a commit to the diwenne's projects Team on Vercel. A member of the Team first needs to authorize it. |
fcgoesdev
pushed a commit
to fcgoesdev/openreply
that referenced
this pull request
Aug 7, 2026
LegalShell hardcoded dark-theme colors (text-white, text-zinc-300, text-cyan-200, border-white/10) while globals.css defines a light theme (--color-background: #ffffff). The result is text rendered white-on-white. Measured contrast against #ffffff, before -> after: heading text-white 1.00:1 -> text-foreground 17.7:1 body text-zinc-300 1.48:1 -> text-foreground 17.7:1 lede/nav text-zinc-300 1.48:1 -> text-muted 4.83:1 timestamp text-cyan-200 1.25:1 -> text-accent 2.80:1 This matters more than a normal styling bug: /privacy, /terms and /data-deletion are the URLs a Meta app reviewer opens when approving Instagram permissions, and today they render as blank pages. No copy or markup changes -- only class names.
fcgoesdev
force-pushed
the
fix/legal-shell-contrast
branch
from
August 7, 2026 20:26
b601b59 to
7bedc85
Compare
fcgoesdev
added a commit
to fcgoesdev/openreply
that referenced
this pull request
Aug 7, 2026
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.
LegalShellhardcodes dark-theme colors whileapp/globals.cssdefines a light theme, so/privacy,/termsand/data-deletioncurrently render white text on a white background.The bug
app/globals.csssets:components/legal-shell.tsxstyles its content withtext-white,text-zinc-300,text-cyan-200andborder-white/10— values that assume a dark background.Measured contrast against
#ffffff:<h1>text-white— 1.00:1text-foreground— 17.7:1text-zinc-300— 1.48:1text-foreground— 17.7:1text-zinc-300— 1.48:1text-muted— 4.83:1text-cyan-200— 1.25:1text-accent— 2.80:1Why this one matters
These three routes are what a Meta app reviewer opens when approving Instagram permissions — they're the URLs you paste into the privacy policy / terms / data deletion fields. Right now they look like blank pages.
I ran into this setting up a self-hosted deployment; the same token changes are running there now.
Scope
Class names only — no copy changes, no markup changes, no new tokens. All four tokens already exist in
globals.cssand are used elsewhere in the app (e.g.app/page.tsx,app/(dashboard)/overview/page.tsx).Two things I deliberately left alone
1.
text-accentis still below WCAG AA.#f97316on white is 2.80:1, under the 4.5:1 required for normal text. I kept it because it matches existing convention —app/login/page.tsx:51uses the sameuppercase … text-accenttreatment. Darkening--color-accentto something like#c2410c(5.18:1) would fix it globally, but that's a project-wide design decision, not mine to make in a drive-by PR. Happy to open a separate one if you want it.2. The same mismatch exists in other components.
components/seo-page-shell.tsxandcomponents/public-site-header.tsxalso pairtext-whitewith effectively-white backgrounds (bg-white/[0.035],bg-background/85). I scoped this PR toLegalShellbecause that case is unambiguous and has the review-blocking consequence above.If the light theme in
globals.cssis the intended direction, those components want the same treatment and I can follow up. If it isn't — if the app is meant to be dark andglobals.cssis what drifted — then this PR is the wrong fix and I'd rather hear that. Either way, let me know and I'll adjust.