Skip to content
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: Revert "fix: do not use CONTEXT_MENU flag for tray menu" #24076

Merged
merged 1 commit into from
Jun 11, 2020

Conversation

MarshallOfSound
Copy link
Member

Reverts #23843

As per #24049

Notes: no-notes

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Jun 11, 2020
@MarshallOfSound MarshallOfSound changed the title Revert "fix: do not use CONTEXT_MENU flag for tray menu" fix: Revert "fix: do not use CONTEXT_MENU flag for tray menu" Jun 11, 2020
@ckerr
Copy link
Member

ckerr commented Jun 11, 2020

I don't love this, since it means we're re-breaking a11y. This change is a tradeoff between an important bug for a small set of a11y users vs a visual annoyance for a larger set of users.

Since tray a11y on Windows has been broken for several versions, it's understandable as an interim move -- but we still need to find a better fix for #11587.

@MarshallOfSound MarshallOfSound merged commit c66282a into master Jun 11, 2020
@release-clerk
Copy link

release-clerk bot commented Jun 11, 2020

No Release Notes

@MarshallOfSound MarshallOfSound deleted the revert-23843-fix-tray-accessebility branch June 11, 2020 18:28
sentialx pushed a commit to sentialx/electron that referenced this pull request Jul 30, 2020
sentialx pushed a commit to sentialx/electron that referenced this pull request Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-pr 🌱 PR opened in the last 24 hours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants