[PR #6983] wpf: fix a handful of issues with the wpf control #26827

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

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

State: closed
Merged: Yes


  • send alt/F10 through the control
    We were not listening for WM_SYSKEY{UP,DOWN}
  • extract the actual scancode during WM_CHAR, not the bitfield
    We were accidentally sending some of the additional keypress data in with
    the character event in Win32 Input Mode
  • set default fg/bg to campbell
    The WPF control starts up in PowerShell blue even though it's not typically used
    in PowerShell blue.
  • don't rely on the font to determine wideness
    This is a cross-port of #2928 to the WPF control
  • deterministic shutdown
    In testing, I saw a handful of crashes on teardown because we were not shutting
    down the render thread properly.
  • don't pass 10 for the font weight ...
    When Cascadia Code is set, it just looks silly.
  • trigger render when selection is cleared, do it under lock

Fixes #6966.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/6983 **State:** closed **Merged:** Yes --- * send alt/F10 through the control We were not listening for WM_SYSKEY{UP,DOWN} * extract the actual scancode during WM_CHAR, not the bitfield We were accidentally sending some of the additional keypress data in with the character event in Win32 Input Mode * set default fg/bg to campbell The WPF control starts up in PowerShell blue even though it's not typically used in PowerShell blue. * don't rely on the font to determine wideness This is a cross-port of #2928 to the WPF control * deterministic shutdown In testing, I saw a handful of crashes on teardown because we were not shutting down the render thread properly. * don't pass 10 for the font weight ... When Cascadia Code is set, it just looks silly. * trigger render when selection is cleared, do it under lock Fixes #6966.
claunia added the pull-request label 2026-01-31 09:18:22 +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#26827