Skip to content

fix(giveback): stop causes-breakdown slices sharing a colour#6317

Closed
tsahimatsliah wants to merge 1 commit into
mainfrom
fix/giveback-breakdown-colors
Closed

fix(giveback): stop causes-breakdown slices sharing a colour#6317
tsahimatsliah wants to merge 1 commit into
mainfrom
fix/giveback-breakdown-colors

Conversation

@tsahimatsliah

@tsahimatsliah tsahimatsliah commented Jul 14, 2026

Copy link
Copy Markdown
Member

Problem

In the "Where the money will go" donut (GivebackCausesBreakdown), two slices could render in near-identical magenta tones — e.g. Open source (cabbage, purple #BA56E1) and Mental health (bacon, pink #F25D82) looked like the same colour.

Cause: SLICE_COLORS listed cabbage and bacon close together (positions 1 and 4), so a typical 4-slice chart painted both.

Fix

Reorder the palette so the earliest slices are maximally distinct hues (purple → yellow → blue → green) and the two magenta-family tones (cabbage, bacon) sit at opposite ends. A typical 4–5 slice chart now never shows both, so no two slices read alike. Colours and tokens are unchanged — only their order.

Notes

  • Pure reorder of the readonly palette array; no logic/type changes.
  • GivebackCausesBreakdown.spec.tsx passes (it doesn't assert specific colours).

🤖 Generated with Claude Code

Preview domain

https://fix-giveback-breakdown-colors.preview.app.daily.dev

cabbage (purple) and bacon (pink) sat next to each other in SLICE_COLORS,
so a 4-slice donut painted two slices in near-identical magenta tones (e.g.
Open source and Mental health). Reorder the palette so the first slices are
maximally distinct hues and the two magenta-family tones sit at opposite
ends, so a typical chart never shows both.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Jul 14, 2026 10:32am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Jul 14, 2026 10:32am

Request Review

@tsahimatsliah

Copy link
Copy Markdown
Member Author

Superseded by #6318, which combines this colour fix with the leaderboard (#6316) into one clean PR off main.

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