Skip to content

fix(bracket+bet): heavier gold ring on Top 8 3rds + dark text on bet CTA#288

Merged
0800tim merged 1 commit into
mainfrom
style/thirds-selected-and-bet-cta
Jun 5, 2026
Merged

fix(bracket+bet): heavier gold ring on Top 8 3rds + dark text on bet CTA#288
0800tim merged 1 commit into
mainfrom
style/thirds-selected-and-bet-cta

Conversation

@0800tim

@0800tim 0800tim commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Two visual bugs Tim flagged on 2026-06-05.

Top 8 3rd-placed teams (/world-cup-2026, thirds tab): selected/unselected contrast was too subtle. The 3px inset gold ring + rgba(.., 0.6) border read as a faint outline, not as an unambiguous "this one is picked" affordance. Bumped to a 5px solid-gold inset with a soft inner highlight + larger outer glow (24px / 0.55 alpha vs 14px / 0.35).

/the-bet bottom CTA: the "Pick your bracket to enter →" pill was rendering gold text on a gold pill (invisible). .vt-bet-cta-primary set color: #15151a but .vt-bet-body a higher in the same file is (0,1,1) specificity vs (0,1,0) and was winning the cascade. Added !important to the CTA's color rule, matching the pattern .vt-bet-cta-ghost already uses for the same reason.

CSS-only, no logic touched.

…t on bet CTA

Two visual bugs Tim flagged on 2026-06-05:

* The selected/unselected contrast on the Top 8 3rd-placed teams tiles
  was too subtle. The 3px inset gold ring + rgba(.., 0.6) border read
  as a faint outline next to the unpicked tiles, not as an unambiguous
  "this one is picked" affordance. Bumped to a 5px solid-gold inset
  with a 1px soft inner highlight + larger outer glow (24px / 0.55
  alpha vs 14px / 0.35). The picked tile now visibly lifts.

* /the-bet bottom CTA ("Pick your bracket to enter →") was rendering
  gold text on a gold pill. The pill's class .vt-bet-cta-primary set
  color: #15151a but .vt-bet-body a (same file, body-paragraph link
  styling) is (0,1,1) specificity vs (0,1,0) and was winning the
  cascade. Added !important to the CTA's color rule, matching the
  pattern .vt-bet-cta-ghost already uses for the same reason.

Refs: sessions/2026-06-05_orchestrator_bracket-save-on-exit.md
Signed-off-by: Tim Thomas <0800tim@gmail.com>
@0800tim 0800tim merged commit 5cf42e8 into main Jun 5, 2026
11 of 12 checks passed
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

DRY-RUN — this verdict is informational; CI is not blocked.

Auto-triage: GREEN — auto-triage clear

Risk score: 0/100

Metric Value
Files changed 2
Lines added 13
Lines removed 5
Apps touched apps/web
New dependencies 0
New 3rd-party hosts 0

No flags raised by the automated scanners. A human reviewer will still take a look.

Labels applied: area:web, auto-triage:green

Posted by @vtorn/pr-triage-bot. How this works: docs/security/01-pr-triage-process.md. Disagree with the verdict? Comment /triage override <reason> and a maintainer will re-review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant