[PR #17715] Add elipses back in to some command names #31334

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

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

State: closed
Merged: Yes


In #16886, the key for the nested action got renamed from Split Pane... to Split pane. This accidentally caused a collision because now there's two actions with the same name! The settings model then prefers the user's action over the one defined in defaults.json, thus completely hiding the nested version.

I tried to balance the stylistic recommendations from #16846 (mainly this comment since it gave some excellent examples) while trying to maintain muscle memory as much as possible (with similar substring sequences). There was also one case where we still used "the tab" so I removed the "the" for consistency.

Side effect of #16886 which closed #16846
Closes #17294, #17684

**Original Pull Request:** https://github.com/microsoft/terminal/pull/17715 **State:** closed **Merged:** Yes --- In #16886, the key for the nested action got renamed from `Split Pane...` to `Split pane`. This accidentally caused a collision because now there's two actions with the same name! The settings model then prefers the user's action over the one defined in defaults.json, thus completely hiding the nested version. I tried to balance the stylistic recommendations from #16846 (mainly [this comment](https://github.com/microsoft/terminal/issues/16846#issuecomment-2005007519) since it gave some excellent examples) while trying to maintain muscle memory as much as possible (with similar substring sequences). There was also one case where we still used "the tab" so I removed the "the" for consistency. Side effect of #16886 which closed #16846 Closes #17294, #17684
claunia added the pull-request label 2026-01-31 09:46:38 +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#31334