Skip to content

fix(ui): avoid duplicate Claude working indicators#78

Merged
simion merged 1 commit into
simion:mainfrom
Ehtasham-Yasin:fix-duplicate-working-indicator
Jul 11, 2026
Merged

fix(ui): avoid duplicate Claude working indicators#78
simion merged 1 commit into
simion:mainfrom
Ehtasham-Yasin:fix-duplicate-working-indicator

Conversation

@Ehtasham-Yasin

Copy link
Copy Markdown
Contributor

Summary

Closes #74.

When Termic's opt-in working indicator is visible, Claude's raw leading status glyph is now hidden from the displayed tab title. This avoids showing two separate working indicators for the same state.

Changes

  • Added a small title-formatting helper for Claude live terminal titles
  • Hide Claude's leading idle/spinner glyph only when Termic's own working badge is actually visible
  • Applied the behavior consistently in the sidebar and top tab bar
  • Preserved raw titles, custom titles, and behavior when the working indicator is disabled
  • Added focused unit tests for idle and Braille spinner title prefixes

Validation

  • npm test — 220 tests passed
  • npm run build — passed
  • git diff --cached --check — passed

@simion

simion commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Thanks! will manually test and verify tonight. Trying to remember what issues i had with this cleanup the last time i implemented it and ended up reverting it.

I think i kept those glyphs for more reliable work done detection. If we do the detection on the original non formatted title, all good.

@simion

simion commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Thanks @Ehtasham-Yasin, this is a clean fix. Verified the raw title is still stored verbatim (setTabLiveTitle) and the working-state detection reads the unmodified title, so the activity indicator is untouched, the formatting is display-only. Tests (220) and build pass. Merging now; it'll ship in the next release. Appreciate the well-scoped PR and the unit tests.

@simion simion merged commit 5959f05 into simion:main Jul 11, 2026
2 checks passed
@Ehtasham-Yasin Ehtasham-Yasin deleted the fix-duplicate-working-indicator branch July 11, 2026 14:11
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.

Sidebar shows two different, unexplained "Claude is working" indicators on the same tab row

2 participants