New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: consider dock space when showing menu #35194
Conversation
Release Notes Persisted
|
I have automatically backported this PR to "18-x-y", please check out #35198 |
I have automatically backported this PR to "19-x-y", please check out #35199 |
I have automatically backported this PR to "20-x-y", please check out #35200 |
Is there any chance this broke again with Electron 22? Seeing it again in VS Code: //cc @deepak1556 |
Seems flaky, now I cannot reproduce anymore, but will keep an eye open. |
Description of Change
Use
visibleFrame
instead offrame
when computing edge of screen, the former excludes the dock and menubar from the frame.Close #14261.
Checklist
npm test
passesRelease Notes
Notes: Fix context menu not showing all items on macOS when dock is not hidden.