Add "Context menu style" setting#2903
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Needs human review This PR introduces a new user-facing feature (context menu style setting) with new UI and runtime behavior changes that determine how context menus are displayed. New features with user-facing settings warrant human review. You can customize Macroscope's approvability policy. Learn more. |
What Changed
Why
Electron context menus look ugly. For example vscode uses custom context menus on platforms except for macOS even when
menuStyleis set tonative.UI Changes
Before:

After:

Settings:

Checklist
Note
Cursor Bugbot is generating a summary for commit b30de10. Configure here.
Note
Add "Context menu style" setting with native, custom, and default options
ContextMenuStyletype ('default','native','custom') to settings.ts and includescontextMenuStyleinClientSettingsSchemawith a'default'fallback.contextMenu.showin localApi.ts to read the style setting and route to the native desktop menu (desktopBridge.showContextMenu) or the in-app fallback accordingly;'default'uses native only on macOS.showContextMenureturnnullinstead of propagating.Macroscope summarized b30de10.