Skip to content

Menu bar: 0 t/s instead of a dash; the numbers-only dot breathes - #345

Merged
fylorn merged 1 commit into
devfrom
menubar-zero-rate-breathing-dot
Oct 19, 2026
Merged

fylorn merged 1 commit into
devfrom
menubar-zero-rate-breathing-dot

Conversation

@fylorn

@fylorn fylorn commented Oct 19, 2026

Copy link
Copy Markdown
Contributor
  • Speed line: shows 0 t/s when no request is running and when running requests have not produced a token yet, instead of — t/s. rate_short now takes a plain u32. The menu's in-flight rows are unchanged (no speed before the first token).
  • Numbers-only dot: breathes while requests run. A Core Animation mask over the dot fades it between full and 25% opacity (ease-in-out, 1 s each way), so the app does no per-frame work, same as the glow on the mark. It is removed while the menu is open and stays solid with Reduce Motion. The glow and the breathing share mask_around for the opaque sides of the mask.

Checked on a live status item with a temporary probe of the presentation layers: the dot's opacity cycles 1.0 → 0.25 → 1.0 every 2 s, and the glow on the mark still sweeps. Clippy and the menu bar tests pass.

🤖 Generated with Claude Code

The speed line reads 0 t/s both when nothing is running and when running
requests have not produced a token yet; it no longer shows "— t/s".

In the numbers-only style the in-flight dot now breathes: a Core Animation
mask on the dot fades it between full and 25% opacity with ease-in-out,
1 s each way, sharing the glow's begin time. Like the glow it is taken off
while the menu is open and stays solid when Reduce Motion is on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fylorn
fylorn merged commit 80e1335 into dev Oct 19, 2026
6 checks passed
@fylorn
fylorn deleted the menubar-zero-rate-breathing-dot branch October 19, 2026 02:10
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant