Drop idle time badge in compact view#33
Merged
Merged
Conversation
The compact layout already trades richness for density; surfacing "2m" / "1h" / "3d" next to a session row added noise that didn't help the at-a-glance scan. Working sessions get the spinner, Waiting sessions get the bell, and Idle sessions now get just the moon glyph — three distinct shapes, no number to read. The first 3 seconds after a transition still show the spinner so the glyph doesn't snap during a quick Working → Idle flip.
Contributor
There was a problem hiding this comment.
Pull request overview
Simplifies the compact sidebar view by removing the idle time badge (e.g. 2m, 1h) next to Idle sessions, replacing it with just the moon glyph for at-a-glance consistency with Working (spinner) and Waiting (bell) statuses. A short 3-second spinner is still shown immediately after entering Idle to avoid abrupt glyph changes during Working → Idle transitions. The tabs detail line is intentionally unchanged.
Changes:
- Replace
format_activity_compactcall with inline moon glyph (or spinner during the first 3 seconds of idle) in compact session rendering. - Update surrounding comment to document the new glyph-only behavior.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
In
ViewMode::Compact, Idle sessions used to show a2m/1h/3dbadge next to the row. That extra number didn't help the at-a-glance scan compact mode is for. Now Idle just shows the moon glyph, matching the shape of Working (spinner) and Waiting (bell).The first 3 seconds after a transition still render the spinner, so the glyph doesn't snap during a quick Working → Idle flip.
Tabs detail line (
format_activity_compactatsidebar.rs:804) is unchanged — it still shows the time, because that bar exists specifically to give a focused-session readout.Test plan
cargo test— 177 passedXm)🤖 Generated with Claude Code