Skip to content

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

Merged
wan9chi merged 2 commits into
mainfrom
claude/fix-issue-264-Ee8Hd
Mar 18, 2026
Merged

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

Conversation

@wan9chi

@wan9chi wan9chi commented Mar 18, 2026

Copy link
Copy Markdown
Member

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

wan9chi commented Mar 18, 2026

Copy link
Copy Markdown
Member Author

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

@wan9chi wan9chi 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>
@wan9chi wan9chi force-pushed the claude/fix-issue-264-Ee8Hd branch from c279b6e to 60c92b0 Compare March 18, 2026 10:40

wan9chi commented Mar 18, 2026

Copy link
Copy Markdown
Member Author

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.

@wan9chi wan9chi merged commit afec83c into main Mar 18, 2026
9 checks passed
@wan9chi wan9chi 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