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: do not use CONTEXT_MENU flag for tray menu #23843
Conversation
1285731
to
c8b991c
Compare
Release Notes Persisted
|
I was unable to backport this PR to "7-3-x" cleanly; |
I was unable to backport this PR to "8-x-y" cleanly; |
I have automatically backported this PR to "10-x-y", please check out #23879 |
I have automatically backported this PR to "9-x-y", please check out #23880 |
This reverts commit b809d10.
…)" (electron#24076) This reverts commit b809d10.
…)" (electron#24076) This reverts commit b809d10.
Description of Change
Close #11587.
Tray menu does not popup on a window, so it should not use
CONTEXT_MENU
flag otherwise some things would not work. The workaround used in #15302 stopped working after some Chromium upgrades.This PR makes our tray menu code aligned with Chromium
status_icon_win.cc
.Release Notes
Notes: Fix tray menu on Windows not keyboard navigable.