Reorder menu bar toggles and clarify their labels#240
Merged
Conversation
Move the per-app enable/disable toggle directly under "Enable Globally" so both scope controls sit together. Rename "Show Indicator" to "Show Cotabby Indicator" and relabel the multi-line toggle to "Allow Multi-line Suggestions", moving it up beneath the indicator toggle next to the other behavior switches instead of trailing the engine/model/length pickers.
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
Tightens the menu bar quick-controls layout. The per-app enable/disable toggle now sits directly beneath Enable Globally so the two scope controls are adjacent. Show Indicator is renamed to Show Cotabby Indicator, and the multi-line switch is relabeled Allow Multi-line Suggestions and moved up next to the other behavior toggles instead of trailing the engine/model/length pickers.
Note: the per-app toggle (
Enable in <app>) is conditional on a focused non-terminal external app, so it only renders when Cotabby has tracked one — it was never removed, just positioned below the clipboard toggle.Validation
UI-only change to existing toggles; behavior of each setting is unchanged.
Linked issues
None.
Risk / rollout notes
Pure presentation change — toggle order and labels only. No settings, schema, or persistence changes.
Greptile Summary
This PR reorders the toggle controls in the menu bar panel and clarifies two toggle labels. No bindings, settings persistence, or behavioral logic is changed.
Confidence Score: 5/5
Safe to merge — purely reorders existing toggles and updates two labels with no functional changes.
Every toggle retains its original binding and behavior. The two renamed labels are purely cosmetic strings. The new ordering groups scope controls together and places all toggles before the pickers, which is a sensible UX improvement with no risk of breakage.
No files require special attention.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Enable Globally] --> B{latestExternalApplication\nnon-terminal?} B -- yes --> C[Enable in app] B -- no --> D[Include Clipboard Context] C --> D D --> E[Show Cotabby Indicator] E --> F[Allow Multi-line Suggestions] F --> G[Engine picker] G --> H{selectedEngine ==\nappleIntelligence\n&& unavailable?} H -- yes --> I[Availability warning] H -- no --> J{supportsLocal\nModelManagement?} I --> J J -- yes --> K[Model picker] J -- no --> L[Length picker] K --> LReviews (1): Last reviewed commit: "Reorder menu bar toggles and clarify the..." | Re-trigger Greptile