feat(ui): toolbar trim — collapse Configure/Clone/Rename into More ▾#16
Merged
Conversation
User feedback after merging the redesign: "too many buttons" in the top toolbar. Counting per-pill: Start | New + Import | Configure + Clone + Rename | Delete — seven persistent toolbar buttons for what in practice fall into three usage tiers: - Primary daily: Start, New - Common: Import, Delete - Occasional: Configure, Clone, Rename This PR consolidates the third tier into a single "More ▾" overflow button that pops a popup menu with the three items. Visible button count drops 7 → 5; the three actions are still one click + one tap away, and their @IBAction selectors / keyboard shortcuts / menu-bar entries all keep working unchanged because the menu items target the exact same selectors the buttons did. New toolbar: [▶ Start] [+ New | ↧ Import] [More ▾] [🗑 Delete] Implementation: - `makeMoreActionsButton()` builds the overflow trigger. - `showMoreActionsMenu(_:)` pops the menu and propagates each wrapped button's `isEnabled` state to its menu item, so a disabled Configure stays disabled in the menu (same enabled rules `updateButtonStates` already maintains). - The Configure / Clone / Rename buttons are still `@IBOutlet` bound — only their pill placement changed. Removing them from any prior pill container before the new layout pass prevents ghost duplicates if the layout function fires twice. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Why
User feedback after the merged redesign: "too many buttons".
Counting the current toolbar by usage tier:
The Occasional tier was eating three persistent toolbar slots for actions taken… occasionally.
What changed
Configure / Clone / Rename collapse into a single More ▾ overflow button. Tap it → popup menu with the three items.
Before:
```
[▶ Start] [+ New | ↧ Import] [⚙ Configure | ⎘ Clone | ✎ Rename] [🗑 Delete]
7 visible buttons
```
After:
```
[▶ Start] [+ New | ↧ Import] [More ▾] [🗑 Delete]
5 visible buttons
```
How the action paths stay intact
The Configure / Clone / Rename buttons are still `@IBOutlet` bound — only their toolbar placement changed. Each menu item fires the same `@IBAction` selector the original button did, so:
Test plan
🤖 Generated with Claude Code