fix: updated Alt detection to explicitly exclude AltGraph/AltGr#49778
fix: updated Alt detection to explicitly exclude AltGraph/AltGr#49778jkleinsc merged 2 commits intoelectron:mainfrom
Conversation
|
@darthvader58 can you rebase your PR with the latest from main? That should fix the lint job failure. |
|
@darthvader58 we require commits to be signed before we can merge this PR: https://www.electronjs.org/docs/latest/development/pull-requests#commit-signing. Can you sign your commits? |
|
@jkleinsc done |
|
@darthvader58 can you rebase one more time? That should resolve the lint CI error. |
@jkleinsc should be good now. |
|
Congrats on merging your first pull request! 🎉🎉🎉 |
|
Release Notes Persisted
|
Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
|
I have automatically backported this PR to "39-x-y", please check out #49914 |
|
I have automatically backported this PR to "40-x-y", please check out #49915 |
|
I have automatically backported this PR to "41-x-y", please check out #49916 |
fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778) Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778) Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778) Updated Alt detection to explicitly exclude AltGraph/AltGr Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
…tron#49778) Updated Alt detection to explicitly exclude AltGraph/AltGr
Description of Change
AltGrkey wrongly interpreted asAlt#49737Fixed Linux menu-bar focus behavior so
AltGr(AltGraph) is no longer treated as a singleAlt. Previously, pressing AltGr could trigger menu-bar activation/toggling (especially with auto-hide RootView to explicitly excludeAltGr/AltGraphkey events. Also added a Linux regression test in spec/api-browser-window-spec.ts to verify that pressingAltGrdoes not toggle auto-hide menu bar visibility.Checklist
npm testpassesRelease Notes
Notes: Fixed an issue where pressing AltGr could activate or focus the menu bar.