[PR #1436] Fixed #521 - AltGr combinations not working #24611

Open
opened 2026-01-31 09:04:20 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/1436

State: closed
Merged: Yes


Summary of the Pull Request

Let's start fixing #521 and the currently broken state of AltGr combinations, as it heavily affects many non-US users of this project.

PR Checklist

  • Closes iTerm color schemes once more are failing to parse colors (#521)
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #521

Detailed Description of the Pull Request / Additional comments

I'm about 100% sure that this PR is not going to fix #521 by itself, but it's a start. 🙂
If others wan't to contribute to this PR, please feel free to do so. I'll gladly grant push permissions to my fork.

If you want to view the diff here on GitHub I recommend enabling "Hide whitespace changes" in the diff viewer.

Validation Steps Performed

This PR fixes all immediate/obvious issues in regards to AltGr for me in all types of shells I know.
I think it's fairly easy to see why it works, since I simply delegate the handling to WM_CHAR instead.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/1436 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Let's start fixing #521 and the currently broken state of AltGr combinations, as it heavily affects many non-US users of this project. ## PR Checklist * [ ] Closes #521 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [x] Tests added/passed * [ ] Requires documentation to be updated * [ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #521 ## Detailed Description of the Pull Request / Additional comments I'm about 100% sure that this PR is not going to fix #521 by itself, but it's a start. 🙂 If others wan't to contribute to this PR, please feel free to do so. I'll gladly grant push permissions to my fork. If you want to view the diff here on GitHub I recommend enabling "Hide whitespace changes" in the diff viewer. ## Validation Steps Performed This PR fixes all immediate/obvious issues in regards to AltGr for me in all types of shells I know. I think it's fairly easy to see why it works, since I simply delegate the handling to `WM_CHAR` instead.
claunia added the pull-request label 2026-01-31 09:04:20 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#24611