Visual Error - shortcut CTRL+SHIFT+[dash,equals] #10207

Closed
opened 2026-01-31 02:15:23 +00:00 by claunia · 9 comments
Owner

Originally created by @kellytrinh on GitHub (Aug 18, 2020).

Environment

WT Version 1.1.2233

Steps to reproduce

  1. Have lots of different profiles defined.
  2. In settings.json extend additional shortcuts from the existing 1 to 9 for 0, - (dash), and = (equals) so that CTRL+SHIFT+[key] opens respective profile

Expected behavior

In the menu for opening a tab/profile, the shortcut for extended shortcuts should appear as logically expected

Actual behavior

In the menu for opening a tab/profile, the shortcut newly defined 0 is appear next to relevant profile but the shortcut for dash and equals do not appear.

Originally created by @kellytrinh on GitHub (Aug 18, 2020). <!-- 🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨 I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING: 1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement. 2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement. 3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number). 4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement. 5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement. All good? Then proceed! --> <!-- This bug tracker is monitored by Windows Terminal development team and other technical folks. **Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues**. Instead, send dumps/traces to secure@microsoft.com, referencing this GitHub issue. If this is an application crash, please also provide a Feedback Hub submission link so we can find your diagnostic data on the backend. Use the category "Apps > Windows Terminal (Preview)" and choose "Share My Feedback" after submission to get the link. Please use this form and describe your issue, concisely but precisely, with as much detail as possible. --> # Environment ```none WT Version 1.1.2233 ``` # Steps to reproduce 1. Have lots of different profiles defined. 2. In `settings.json` extend additional shortcuts from the existing 1 to 9 for 0, - (dash), and = (equals) so that CTRL+SHIFT+[key] opens respective profile # Expected behavior In the menu for opening a tab/profile, the shortcut for extended shortcuts should appear as logically expected # Actual behavior In the menu for opening a tab/profile, the shortcut newly defined 0 is appear next to relevant profile but the shortcut for dash and equals do not appear.
Author
Owner

@zadjii-msft commented on GitHub (Aug 18, 2020):

Could you share the actual json you've added for these keybindings?

@zadjii-msft commented on GitHub (Aug 18, 2020): Could you share the actual json you've added for these keybindings?
Author
Owner

@kellytrinh commented on GitHub (Aug 18, 2020):

The relevant lines below; the shortcut themselves do work and I can open the tab/profiles with them fine.

        { "command": { "action": "newTab", "index": 7 }, "keys": "ctrl+shift+9" },
        { "command": { "action": "newTab", "index": 8 }, "keys": "ctrl+shift+0" },        
        { "command": { "action": "newTab", "index": 9 }, "keys": "ctrl+shift+-" },                
        { "command": { "action": "newTab", "index": 10 }, "keys": "ctrl+shift+=" },                

@kellytrinh commented on GitHub (Aug 18, 2020): The relevant lines below; the shortcut themselves do work and I can open the tab/profiles with them fine. ``` { "command": { "action": "newTab", "index": 7 }, "keys": "ctrl+shift+9" }, { "command": { "action": "newTab", "index": 8 }, "keys": "ctrl+shift+0" }, { "command": { "action": "newTab", "index": 9 }, "keys": "ctrl+shift+-" }, { "command": { "action": "newTab", "index": 10 }, "keys": "ctrl+shift+=" }, ```
Author
Owner

@DHowett commented on GitHub (Aug 18, 2020):

It looks like we only actually do this for the first 9 profiles:

93d266925c/src/cascadia/TerminalApp/TerminalPage.cpp (L465-L468)

This is probably because we had 9 hardcoded "switch to tab" commands and it was not possible to add a custom index when we first wrote this code 😄

@DHowett commented on GitHub (Aug 18, 2020): It looks like we only actually do this for the first 9 profiles: https://github.com/microsoft/terminal/blob/93d266925cfce92c1f321c43e10809d624cd2b94/src/cascadia/TerminalApp/TerminalPage.cpp#L465-L468 This is probably because we had 9 hardcoded "switch to tab" commands and it was not possible to add a custom index when we first wrote this code :smile:
Author
Owner

@zadjii-msft commented on GitHub (Aug 18, 2020):

Ho boy would you look at that. Yep, we should fix that

@zadjii-msft commented on GitHub (Aug 18, 2020): Ho boy would you look at that. Yep, we should fix that
Author
Owner

@kellytrinh commented on GitHub (Aug 19, 2020):

Just noticed - for some reason there seems to be some bug in the shortcut handling - with the JSON snippet above; CTRL+SHIFT+9 works. So does CTRL+SHIFT+[dash]. But the one in the middle CTRL+SHIFT+0 oddly does nothing.

So there is two bugs, the OP with [dash] [equals] shortcuts not appears and additionally the sequence CTRL+SHIFT+0 not being handled.

@kellytrinh commented on GitHub (Aug 19, 2020): Just noticed - for some reason there seems to be some bug in the shortcut handling - with the JSON snippet above; CTRL+SHIFT+9 works. So does CTRL+SHIFT+[dash]. But the one in the middle CTRL+SHIFT+0 oddly does nothing. So there is two bugs, the OP with [dash] [equals] shortcuts not appears and additionally the sequence CTRL+SHIFT+0 not being handled.
Author
Owner

@DHowett commented on GitHub (Aug 19, 2020):

If you have multiple keyboard layouts enabled, Ctrl+Shift+0 is reserved by Windows as a keyboard layout switch command.

@DHowett commented on GitHub (Aug 19, 2020): If you have multiple keyboard layouts enabled, Ctrl+Shift+0 is reserved by Windows as a keyboard layout switch command.
Author
Owner

@kellytrinh commented on GitHub (Aug 19, 2020):

If you mean multiple languages - no I don't have anything like that installed.

Not sure what other layouts you could mean but happy to look into if you give me a pointer on where to check the windows setting.

Anyway; if you are sure this is not a bug and expected behaviour due to windows outside; then can ignore.

@kellytrinh commented on GitHub (Aug 19, 2020): If you mean multiple languages - no I don't have anything like that installed. Not sure what other layouts you could mean but happy to look into if you give me a pointer on where to check the windows setting. Anyway; if you are sure this is not a bug and expected behaviour due to windows outside; then can ignore.
Author
Owner

@ghost commented on GitHub (Aug 26, 2020):

:tada:This issue was addressed in #7344, which has now been successfully released as Windows Terminal v1.2.2381.0.🎉

Handy links:

@ghost commented on GitHub (Aug 26, 2020): :tada:This issue was addressed in #7344, which has now been successfully released as `Windows Terminal v1.2.2381.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.2.2381.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Aug 26, 2020):

:tada:This issue was addressed in #7344, which has now been successfully released as Windows Terminal Preview v1.3.2382.0.🎉

Handy links:

@ghost commented on GitHub (Aug 26, 2020): :tada:This issue was addressed in #7344, which has now been successfully released as `Windows Terminal Preview v1.3.2382.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.3.2382.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#10207