🤖 Hide old workspaces in sidebar with expandable section #361
+285
−5
Conversation
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
- Hide workspaces inactive for >24 hours by default per project - Add 'Show N older workspaces' toggle button with count - Persist expansion state in localStorage - Fully reversible expansion/collapse - Includes comprehensive unit tests Improves sidebar UX by reducing clutter from stale workspaces while keeping them accessible when needed.
- Change text from 'Show N older workspaces' to 'Show workspaces older than 1 day (N)' - Add formatOldWorkspaceThreshold() for human-readable cutoff display - Enhance button styling with better colors, border-top separator, and spacing - Use CSS classes for arrow and count styling - Fix boundary condition: workspaces at exactly 24 hours are now correctly marked as old Design improvements: - Clearer visual separation with border-top - Better hover states with subtle background change - Count displayed in parentheses as secondary info - More professional, less cluttered appearance
- Text stays constant: 'OLDER THAN 1 DAY (N)' - Only arrow changes: ▶ when collapsed, ▼ when expanded - Styled as section header with uppercase text - Better visual hierarchy with consistent spacing - Added aria-expanded for accessibility
- Arrow moved to right side and rotates in place - Text 'Older than 1 day (N)' stays constant on left - Arrow rotates 90° from ▶ to ▼ when expanded - Restored normal font weight (not uppercase) - Perfect symmetry: all elements stay in same position
- When all workspaces are old, show the most recent one - Prevents empty workspace list in sidebar - Updated tests to cover this behavior - Improves UX by ensuring projects always show their most recent workspace
- Changed from Set to Record<string, boolean> for clearer per-project tracking - Now properly scoped: each project path has its own expansion state - Simplified toggle logic
- Extract workspace rendering into renderWorkspace helper - Reduce nesting depth by restructuring JSX - Eliminate duplicate WorkspaceListItem code - Move WorkspacesContainer outside IIFE - Clearer separation of concerns
ammario
approved these changes
Oct 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Improves sidebar UX by hiding workspaces that haven't been used in more than 24 hours, while keeping them accessible via an expandable section per project.
Changes
New utility:
workspaceFiltering.ts: Partitions workspaces into recent (<24h) and old (≥24h) based on recency timestampsComponent updates:
workspaceRecencythrough component treeState management:
expandedOldWorkspaces)Tests:
UX Flow
Generated with
cmux