[PR #5552] [MERGED] Fix Ctrl+Alt not being treated as a substitute for AltGr anymore #26367

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5552
Author: @lhecker
Created: 4/24/2020
Status: Merged
Merged: 4/28/2020
Merged by: @DHowett-MSFT

Base: masterHead: fix-5525-altgr-alt-ctrl


📝 Commits (1)

  • bfa16c6 Fix Ctrl+Alt not being treated as a substitute for AltGr anymore

📊 Changes

2 files changed (+12 additions, -12 deletions)

View changed files

📝 src/terminal/adapter/ut_adapter/inputTest.cpp (+2 -2)
📝 src/terminal/input/terminalInput.cpp (+10 -10)

📄 Description

Summary of the Pull Request

This PR fixes #5525 by re-adding range checks that where erroneously removed in a9c9714.

PR Checklist

Validation Steps Performed

  • Enabled a German keyboard layout
  • Entered <, +, 7, 8, 9, 0 while holding either Alt+Ctrl or AltGr and...
  • Ensuring that both produce |, ~, {, [, ], }

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/5552 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 4/24/2020 **Status:** ✅ Merged **Merged:** 4/28/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `fix-5525-altgr-alt-ctrl` --- ### 📝 Commits (1) - [`bfa16c6`](https://github.com/microsoft/terminal/commit/bfa16c64f26b83b2dd51bc9bdb135d4896d61197) Fix Ctrl+Alt not being treated as a substitute for AltGr anymore ### 📊 Changes **2 files changed** (+12 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `src/terminal/adapter/ut_adapter/inputTest.cpp` (+2 -2) 📝 `src/terminal/input/terminalInput.cpp` (+10 -10) </details> ### 📄 Description ## Summary of the Pull Request This PR fixes #5525 by re-adding range checks that where erroneously removed in a9c9714. ## PR Checklist * [x] Closes #5525 * [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 * [x] 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: #5525 ## Validation Steps Performed * Enabled a German keyboard layout * Entered `<`, `+`, `7`, `8`, `9`, `0` while holding either Alt+Ctrl or AltGr and... * Ensuring that both produce `|`, `~`, `{`, `[`, `]`, `}` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:15:38 +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#26367