Skip to content

fix(app): use timeline accent for unread divider#457

Merged
brsbl merged 2 commits into
mainfrom
bb/use-non-error-color-thr_uriedjjmrw
Jul 1, 2026
Merged

fix(app): use timeline accent for unread divider#457
brsbl merged 2 commits into
mainfrom
bb/use-non-error-color-thr_uriedjjmrw

Conversation

@brsbl

@brsbl brsbl commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a semantic timeline-accent theme token for the timeline
  • map it to bb’s existing soft paper-blue timeline tint in the default theme and to each built-in palette’s blue/cyan accent
  • keep file-accent aliased to timeline-accent so existing file-path styling stays aligned
  • use timeline-accent for timeline title accents and the unread divider instead of destructive red

Rationale

  • Red/destructive remains reserved for errors and destructive states.
  • primary is not the right semantic fit here: in the default bb theme it is neutral, and in custom palettes it is the general product/control accent.
  • The timeline already has one chromatic blue/cyan affordance for link-like timeline targets, so timeline-accent makes the unread marker match that visual language without borrowing an error token or a file-specific name.

Validation

  • pnpm exec turbo run test --filter=@bb/app -- src/components/ui/theme.test.ts
  • pnpm exec turbo run typecheck --filter=@bb/app

@brsbl brsbl changed the title fix(app): use link accent for unread divider fix(app): use timeline accent for unread divider Jul 1, 2026
@brsbl brsbl merged commit f9a7646 into main Jul 1, 2026
6 checks passed
@brsbl brsbl deleted the bb/use-non-error-color-thr_uriedjjmrw branch July 1, 2026 19:03
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