Skip to content

Fix: distinct color per confidence level (4 ≠ 5, 2 ≠ 1)#5

Merged
guuse merged 1 commit into
mainfrom
fix/confidence-colors
Jun 2, 2026
Merged

Fix: distinct color per confidence level (4 ≠ 5, 2 ≠ 1)#5
guuse merged 1 commit into
mainfrom
fix/confidence-colors

Conversation

@guuse

@guuse guuse commented Jun 2, 2026

Copy link
Copy Markdown
Owner

What

Confidence 4 and 5 rendered the same green (both bucketed as "hoog"), and 1/2 the same red — so the badge number was the only difference between them.

Each level now gets a distinct shade along a green → amber → red gradient, in both the timeline blocks and the leftover-sidebar chips:

Conf Color
5 deep green
4 light green
3 amber
2 orange
1 red

1682 tests pass, typecheck + lint clean.

🤖 Generated with Claude Code

Confidence 4 and 5 rendered identical green (both "hoog"), and 1/2 identical red,
so the badge number was the only difference. Give each level its own shade along
a green → amber → red gradient in both the timeline blocks and the leftover
sidebar chips: 5 deep green, 4 light green, 3 amber, 2 orange, 1 red.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 99.21% (🎯 95%) 2668 / 2689
🔵 Statements 98.64% (🎯 95%) 3048 / 3090
🔵 Functions 98.24% (🎯 95%) 784 / 798
🔵 Branches 96.07% (🎯 95%) 1713 / 1783
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/ui/components/DayTimeline.tsx 99.3% 92.42% 100% 100% 141
src/ui/components/LeftoverSidebar.tsx 85.71% 76.47% 88.23% 88.23% 39, 58-59, 113
Generated in workflow #17 for commit 3fc96eb by the Vitest Coverage Report Action

@guuse guuse merged commit e80c435 into main Jun 2, 2026
1 check passed
@guuse guuse deleted the fix/confidence-colors branch June 2, 2026 12:34
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