Skip to content

fix: use neutral symbols for cache hit/miss indicators#268

Merged
branchseer merged 2 commits intomainfrom
claude/fix-issue-264-Ee8Hd
Mar 18, 2026
Merged

fix: use neutral symbols for cache hit/miss indicators#268
branchseer merged 2 commits intomainfrom
claude/fix-issue-264-Ee8Hd

Conversation

@branchseer
Copy link
Member

@branchseer branchseer commented Mar 18, 2026

Replace misleading error-like symbols with neutral ones:

  • Cache hit: ✓ → ◉ (filled circle)
  • Cache miss: ✗ → 〇 (empty circle)
  • Cache miss color: purple/magenta → bright_black

image.png

image.png

Fixes #264

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@branchseer branchseer requested a review from cpojer March 18, 2026 10:34
claude and others added 2 commits March 18, 2026 10:40
Replace misleading error-like symbols with neutral ones:
- Cache hit: ✓ → ◉ (filled circle)
- Cache miss: ✗ → 〇 (empty circle)
- Cache miss color: purple/magenta → bright_black

Fixes #264

https://claude.ai/code/session_01GZYtvRdFsFHWg8ENzPxc4T
Replace 〇 (U+3007, CJK ideographic number zero) with ○ (U+25CB, white
circle) so the cache miss symbol matches the size of the cache hit ◉
(U+25C9, fisheye) — both are now from the same geometric shapes block.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@branchseer branchseer force-pushed the claude/fix-issue-264-Ee8Hd branch from c279b6e to 60c92b0 Compare March 18, 2026 10:40
Copy link
Member Author

branchseer commented Mar 18, 2026

Merge activity

  • Mar 18, 10:41 AM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 18, 10:47 AM UTC: @branchseer merged this pull request with Graphite.

@branchseer branchseer merged commit afec83c into main Mar 18, 2026
9 checks passed
@branchseer branchseer deleted the claude/fix-issue-264-Ee8Hd branch March 18, 2026 10:47
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.

Consider changing the "✗" for cache miss

3 participants