Skip to content

fix: updated Alt detection to explicitly exclude AltGraph/AltGr#49778

Merged
jkleinsc merged 2 commits intoelectron:mainfrom
darthvader58:main
Feb 23, 2026
Merged

fix: updated Alt detection to explicitly exclude AltGraph/AltGr#49778
jkleinsc merged 2 commits intoelectron:mainfrom
darthvader58:main

Conversation

@darthvader58
Copy link
Copy Markdown
Contributor

@darthvader58 darthvader58 commented Feb 11, 2026

Description of Change

  • Fixes bug: Pressing AltGr key wrongly interpreted as Alt #49737
    Fixed Linux menu-bar focus behavior so AltGr (AltGraph) is no longer treated as a single Alt . Previously, pressing AltGr could trigger menu-bar activation/toggling (especially with auto-hide RootView to explicitly exclude AltGr/AltGraph key events. Also added a Linux regression test in spec/api-browser-window-spec.ts to verify that pressing AltGr does not toggle auto-hide menu bar visibility.

Checklist

Release Notes

Notes: Fixed an issue where pressing AltGr could activate or focus the menu bar.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Feb 11, 2026
@jkleinsc jkleinsc added the semver/patch backwards-compatible bug fixes label Feb 12, 2026
@jkleinsc jkleinsc changed the title Updated Alt detection to explicitly exclude AltGraph/AltGr fix: updated Alt detection to explicitly exclude AltGraph/AltGr Feb 12, 2026
@jkleinsc jkleinsc added target/39-x-y PR should also be added to the "39-x-y" branch. target/41-x-y PR should also be added to the "41-x-y" branch. labels Feb 12, 2026
@jkleinsc
Copy link
Copy Markdown
Member

@darthvader58 can you rebase your PR with the latest from main? That should fix the lint job failure.

@jkleinsc jkleinsc added the target/40-x-y PR should also be added to the "40-x-y" branch. label Feb 12, 2026
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Feb 12, 2026
@jkleinsc
Copy link
Copy Markdown
Member

@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?

@darthvader58
Copy link
Copy Markdown
Contributor Author

@jkleinsc done

@jkleinsc
Copy link
Copy Markdown
Member

@darthvader58 can you rebase one more time? That should resolve the lint CI error.

@darthvader58
Copy link
Copy Markdown
Contributor Author

darthvader58 commented Feb 20, 2026

@darthvader58 can you rebase one more time? That should resolve the lint CI error.

@jkleinsc should be good now.

@jkleinsc jkleinsc merged commit 90c9de7 into electron:main Feb 23, 2026
108 of 109 checks passed
@welcome
Copy link
Copy Markdown

welcome bot commented Feb 23, 2026

Congrats on merging your first pull request! 🎉🎉🎉

@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Feb 23, 2026

Release Notes Persisted

Fixed an issue where pressing AltGr could activate or focus the menu bar.

trop bot added a commit that referenced this pull request Feb 23, 2026
Updated Alt detection to explicitly exclude AltGraph/AltGr

Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
trop bot added a commit that referenced this pull request Feb 23, 2026
Updated Alt detection to explicitly exclude AltGraph/AltGr

Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
trop bot added a commit that referenced this pull request Feb 23, 2026
Updated Alt detection to explicitly exclude AltGraph/AltGr

Co-authored-by: Shashwat Raj <65155843+darthvader58@users.noreply.github.com>
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Feb 23, 2026

I have automatically backported this PR to "39-x-y", please check out #49914

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Feb 23, 2026

I have automatically backported this PR to "40-x-y", please check out #49915

@trop trop bot removed the target/39-x-y PR should also be added to the "39-x-y" branch. label Feb 23, 2026
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Feb 23, 2026

I have automatically backported this PR to "41-x-y", please check out #49916

@trop trop bot added in-flight/40-x-y in-flight/41-x-y and removed target/40-x-y PR should also be added to the "40-x-y" branch. target/41-x-y PR should also be added to the "41-x-y" branch. labels Feb 23, 2026
dsanders11 pushed a commit that referenced this pull request Feb 23, 2026
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>
@trop trop bot added merged/41-x-y PR was merged to the "41-x-y" branch. and removed in-flight/41-x-y labels Feb 23, 2026
dsanders11 pushed a commit that referenced this pull request Feb 23, 2026
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>
@trop trop bot added merged/40-x-y PR was merged to the "40-x-y" branch. and removed in-flight/40-x-y labels Feb 23, 2026
dsanders11 pushed a commit that referenced this pull request Feb 23, 2026
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>
@trop trop bot added merged/39-x-y PR was merged to the "39-x-y" branch. and removed in-flight/39-x-y labels Feb 23, 2026
kycutler pushed a commit to kycutler/electron that referenced this pull request Feb 26, 2026
…tron#49778)

Updated Alt detection to explicitly exclude AltGraph/AltGr
clavin added a commit that referenced this pull request Mar 5, 2026
jkleinsc pushed a commit that referenced this pull request Mar 6, 2026
…Gr (#49778)" (#50093)

Revert "fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778)"

This reverts commit 90c9de7.

Ref: #50050
trop bot added a commit that referenced this pull request Mar 6, 2026
…Gr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: clavin <clavin@electronjs.org>
trop bot added a commit that referenced this pull request Mar 6, 2026
…Gr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: clavin <clavin@electronjs.org>
trop bot added a commit that referenced this pull request Mar 6, 2026
…Gr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: clavin <clavin@electronjs.org>
VerteDinde pushed a commit that referenced this pull request Mar 7, 2026
…Gr (#49778)" (#50111)

Revert "fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: clavin <clavin@electronjs.org>
VerteDinde pushed a commit that referenced this pull request Mar 7, 2026
…Gr (#49778)" (#50110)

Revert "fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: clavin <clavin@electronjs.org>
jkleinsc pushed a commit that referenced this pull request Mar 9, 2026
…50109)

Revert "fix: updated Alt detection to explicitly exclude AltGraph/AltGr (#49778)"

This reverts commit 90c9de7.

Ref: #50050

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: clavin <clavin@electronjs.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged/39-x-y PR was merged to the "39-x-y" branch. merged/40-x-y PR was merged to the "40-x-y" branch. merged/41-x-y PR was merged to the "41-x-y" branch. semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Pressing AltGr key wrongly interpreted as Alt

3 participants