microsoft/Terminal

Releasing both pressed Shift keys (left+right) reports only one

Open

#18,223 opened on Nov 20, 2024

View on GitHub
 (3 comments) (0 reactions) (0 assignees)C++ (3,212 forks)batch import
Area-InputHelp WantedIssue-BugPriority-3Product-Terminal

Repository metrics

Stars
 (35,764 stars)
PR merge metrics
 (Avg merge 27d 19h) (24 merged PRs in 30d)

Description

Windows Terminal version

current main

Windows build number

10.0.19045.5011

Other Software

No response

Steps to reproduce

  1. Activate "debug tap" pane.
    • Add the global setting (near the top of your JSON file): 
      "debugFeatures": true
    • Open a new tab while holding down both left and right Alt .
  2. Press and release both Shift keys simultaneously.

Expected Behavior

There are four reports in the "debug tap" pane:

  • The first Shift key was pressed.
  • The second Shift key was pressed.
  • Some Shift key was released.
  • The remaining Shift key was released.

Actual Behavior

Image

There are only three reports in the "debug tap" pane:

  • The first Shift key was pressed.
  • The second Shift key was pressed.
  • Some Shift key was released.

Contributor guide

Research direction
Investigate the keyboard event handling in the terminal core, specifically the logic that processes key releases for modifier keys like Shift. Look for where shift key releases are aggregated or deduplicated.
Tech stack
cpp
Domain
desktop
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++Windows API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50