[codex] Group menu sections - #25
Merged
Merged
Conversation
ygsgdbd
force-pushed
the
codex/group-menu-sections
branch
from
July 21, 2026 09:11
432303e to
229a96d
Compare
ygsgdbd
marked this pull request as ready for review
July 21, 2026 09:21
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
TypeSwitch currently wraps several single menu entries in separate SwiftUI
Sectioncontainers. On macOS, each container renders its own separator, so the lower half of the menu appears fragmented even though the entries belong to a small number of related functional groups.This change follows the grouping pattern already used by ListenBar: titled, dynamic application lists remain
Sections, while ordinary actions are arranged continuously and explicitDividers separate only distinct functional areas.User impact
The application management entries now read as one cohesive block instead of a stack of one-item groups. Settings, update/repository actions, and quit remain clearly separated. Menu labels, ordering, submenus, input method behavior, persistence, and actions are unchanged.
Root cause
ConfiguredAppsView,UnavailableAppsView,IgnoredAppsView,SwitchStatisticsView,SettingsView, andAppInfoVieweach introduced container-levelSections. These sections were useful as code organization but unintentionally became visible separators in the native menu.Changes
MenuBarViewbetween the application list, management, settings, and information areas.Validation
just lintjust test— 97 tests passed with 0 failures after rebasing onto the latestmaingit diff --checkThe deterministic README screenshot generator was also attempted, but it correctly stopped because another TypeSwitch instance was running. No screenshot assets were modified.