[PR #10164] Clear all state tracking nested commands when switching command mode #27943

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

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

State: closed
Merged: Yes


Went for option 2 proposed here:
https://github.com/microsoft/terminal/issues/10140#issuecomment-845193132.

Disabling back space in the nested entry didn't felt as the nicest
solution. Instead now all state that keeps track of nested commands is
cleared when switching beteen modes.

Validation Steps Performed

  • Validated the specified issue is fixed by this change:. now after
    entering a sub command and hitting backspace the palette no longer
    shows the sub command item (here < Select color scheme... ).
  • Validated that switching between all modes (command line, actions, tab
    search & tab switch) still work as expected.
  • Validated as well that all modes still work as expected.

Didn't add unit tests, but happy to try that out if this would be
required.

Closes #10140

**Original Pull Request:** https://github.com/microsoft/terminal/pull/10164 **State:** closed **Merged:** Yes --- Went for option 2 proposed here: https://github.com/microsoft/terminal/issues/10140#issuecomment-845193132. Disabling back space in the nested entry didn't felt as the nicest solution. Instead now all state that keeps track of nested commands is cleared when switching beteen modes. ## Validation Steps Performed - Validated the specified issue is fixed by this change:. now after entering a sub command and hitting backspace the palette no longer shows the sub command item (here `< Select color scheme...` ). - Validated that switching between all modes (command line, actions, tab search & tab switch) still work as expected. - Validated as well that all modes still work as expected. Didn't add unit tests, but happy to try that out if this would be required. Closes #10140
claunia added the pull-request label 2026-01-31 09:25:17 +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#27943