[PR #8044] Enable shortcut while CommandPalette is open #27073

Closed
opened 2026-01-31 09:19:50 +00:00 by claunia · 0 comments
Owner

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

State: closed
Merged: Yes


Summary of the Pull Request

It is maybe not the best way since I had to get all the cases for key handling so I just created for each of them. As a result the code get longer(not optimized). Most difficult thing was Next tab and Previous tab I just could not solve it.

9 commands that couldn't enabled > <

 1. Increase font size               -> could not find VirtualKey for "-"
 2. Decrease font size             -> could not find VirtualKey for "+"
 3. Split pane, split:horizontal -> could not find VirtualKey for "-"
 4. Split pane, split:vertical     -> could not find VirtualKey for "+"
 5. Open default settings        -> could not find VirtualKey for ","
 6. Open settings file               -> could not find VirtualKey for ","
 7. Open new tab dropdown    -> could not resolve keybindings
 8. Next tab                               -> could not resolve keybindings
 9. Previous tab                        -> could not resolve keybindings

References

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

**Original Pull Request:** https://github.com/microsoft/terminal/pull/8044 **State:** closed **Merged:** Yes --- <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request It is maybe not the best way since I had to get all the cases for key handling so I just created for each of them. As a result the code get longer(not optimized). Most difficult thing was Next tab and Previous tab I just could not solve it. ### 9 commands that couldn't enabled > < 1. Increase font size -> could not find VirtualKey for "-" 2. Decrease font size -> could not find VirtualKey for "+" 3. Split pane, split:horizontal -> could not find VirtualKey for "-" 4. Split pane, split:vertical -> could not find VirtualKey for "+" 5. Open default settings -> could not find VirtualKey for "," 6. Open settings file -> could not find VirtualKey for "," 7. Open new tab dropdown -> could not resolve keybindings 8. Next tab -> could not resolve keybindings 9. Previous tab -> could not resolve keybindings <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes #6679 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema 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: #xxx <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed
claunia added the pull-request label 2026-01-31 09:19:50 +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#27073