Skip to content

fix (desktop): menu behavior in the desktop app on windows/linux - #9872

Closed
JRH89 wants to merge 3 commits into
anomalyco:devfrom
JRH89:dev
Closed

fix (desktop): menu behavior in the desktop app on windows/linux#9872
JRH89 wants to merge 3 commits into
anomalyco:devfrom
JRH89:dev

Conversation

@JRH89

@JRH89JRH89 commented Jan 21, 2026•
edited
Loading

Copy link
Copy Markdown

Summary

Fixes#9873
Fixed critical desktop app issues where menus only worked on macOS and windows had rendering problems.

Changes

  • Cross-platform menus: Added menu support for Windows/Linux (previously macOS-only)
  • Window fixes: Resolved window creation conflict and title bar rendering issues
  • Error handling: Added user-friendly error messages for menu actions
  • Configuration: Fixed window sizing and title bar style

Testing

  • Desktop app now starts successfully on Linux
  • Menus work on all platforms (Windows/Linux/macOS)
  • Proper window rendering without conflicts

Fixes the broken desktop app experience.

@github-actions

github-actionsBot commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

github-actionsBot commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@JRH89JRH89 changed the title fix(desktop): resolve cross-platform menu and window issuesFixes #9873bJan 21, 2026
@github-actions

github-actionsBot commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Hey! Your PR title Fixes #9873b doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@JRH89JRH89 changed the title Fixes #9873bFixes #9873Jan 21, 2026
@adamdotdevin

adamdotdevin commented Jan 21, 2026

Copy link
Copy Markdown
Member

@JRH89 could you please add some screenshots?

@JRH89

JRH89 commented Jan 21, 2026

Copy link
Copy Markdown
Author

yes. As I was screenshotting I noticed some menu behavior that had not been covered so I am going back to correct it.

@JRH89JRH89 changed the title Fixes #9873fix: menu behavior in the desktop app on windows/linuxJan 21, 2026
@JRH89

JRH89 commented Jan 21, 2026

Copy link
Copy Markdown
Author

before:
screenshot-2026-01-21_14-57-02
after:
screenshot-2026-01-21_15-02-28

@JRH89

JRH89 commented Jan 22, 2026•
edited
Loading

Copy link
Copy Markdown
Author

@JRH89 could you please add some screenshots?

k i added the screenshots

@JRH89JRH89 changed the title fix: menu behavior in the desktop app on windows/linuxfix (desktop): menu behavior in the desktop app on windows/linuxJan 22, 2026
@JRH89
fix(desktop): resolve cross-platform menu and window issues
a984395
- Fix menus only working on macOS by adding cross-platform support
- Resolve window creation conflict between config and Rust code
- Improve error handling with user feedback for menu actions
- Change title bar style from problematic Overlay to Visible
- Add proper window sizing and fallbacks
- Ensure desktop app works on Windows/Linux/macOS
Fixes broken desktop app experience where menus wouldn't open
and windows had rendering issues.
@JRH89
fix: restore scrollbars in dialogs and provider selection
4564193
- Remove scrollbar hiding from Dialog component to show scrollbars by default
- Add explicit scrollbar styling for provider selection dialog
- Increase dialog height to 700px to show more content and trigger scrollbars
- Add bottom padding to list scroll areas for better spacing
- Remove no-scrollbar classes from layout containers to restore visibility
Fixes issue where provider list and other scrollable content had no visible scrollbars
@JRH89

JRH89 commented Jan 22, 2026

Copy link
Copy Markdown
Author

Hey Adam, I went back and cleaned up the PR.
The version bump and hardcoded paths were temporary local changes I used to get the desktop app running in dev. I forgot to revert them before pushing.
Those have been removed, and the PR now only contains the intended fix.

@github-actionsgithub-actionsBot mentioned this pull request Jan 22, 2026
Closed
@thdxr
thdxrforce-pushed the dev branch 3 times, most recently from f1ae801 to 08fa7f7CompareJanuary 30, 2026 14:37
@github-actions

github-actionsBot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adamdotdevinadamdotdevinAwaiting requested review from adamdotdevin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix: Desktop app menus only work on macOS, windows have rendering issues

2 participants

@JRH89@adamdotdevin