Skip to content

refactor(desktop): remove native MenuBar from main window#4888

Merged
jamesarich merged 1 commit intomainfrom
feat/remove-menubar
Mar 23, 2026
Merged

refactor(desktop): remove native MenuBar from main window#4888
jamesarich merged 1 commit intomainfrom
feat/remove-menubar

Conversation

@jamesarich
Copy link
Copy Markdown
Collaborator

This commit removes the Compose Multiplatform MenuBar implementation from the desktop entry point. The application's navigation and settings access, previously handled via the native system menu, are being streamlined or relocated.

Specific changes include:

  • Removed MenuBar and all its associated submenus (File, View, Navigate, Help) from Main.kt.
  • Removed unused imports related to keyboard shortcuts and window menu components.
  • Deleted the logic for manual navigation stack manipulation within the menu item callbacks.

This commit removes the Compose Multiplatform `MenuBar` implementation from the desktop entry point. The application's navigation and settings access, previously handled via the native system menu, are being streamlined or relocated.

Specific changes include:
- Removed `MenuBar` and all its associated submenus (File, View, Navigate, Help) from `Main.kt`.
- Removed unused imports related to keyboard shortcuts and window menu components.
- Deleted the logic for manual navigation stack manipulation within the menu item callbacks.

Signed-off-by: James Rich <[email protected]>
@github-actions github-actions bot added the enhancement New feature or request label Mar 23, 2026
@jamesarich jamesarich enabled auto-merge March 23, 2026 11:50
@jamesarich jamesarich added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit acb328d Mar 23, 2026
8 checks passed
@jamesarich jamesarich deleted the feat/remove-menubar branch March 23, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant