From dd5744bbf704584c218b1be13526e451f5b713db Mon Sep 17 00:00:00 2001 From: l0ng-ai <24760907+l0ng-ai@users.noreply.github.com> Date: Sat, 8 Aug 2026 19:56:42 +0700 Subject: [PATCH] fix(keybindings): name the switcher after the command, not the home tile The keybindings page read ToggleSwitcher's name off the home screen's string while TogglePalette read its own off the menu bar, so the one action that appears in both places was the one whose name could drift. Point it at CmdSwitchWorkspace, the name the palette shows, and pin it with the assertion the page's other names already carry. --- src/ui/keymap.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/ui/keymap.rs b/src/ui/keymap.rs index 3547e302..a6cb5a2f 100644 --- a/src/ui/keymap.rs +++ b/src/ui/keymap.rs @@ -489,7 +489,7 @@ fn authored_entry(action: &str) -> Option<(CommandGroup, String)> { ), "ToggleSwitcher" => ( CommandGroup::Workspaces, - t(L10nKey::HomeSwitchWorkspace).to_string(), + t(L10nKey::CmdSwitchWorkspace).to_string(), ), "IncreaseFontSize" => ( CommandGroup::View, @@ -997,6 +997,7 @@ mod tests { assert_eq!(action_entry("CloseActiveTab").1, "Close Pane / Tab"); assert_eq!(action_entry("ClearScrollback").1, "Clear Scrollback"); assert_eq!(action_entry("TogglePalette").1, "Command Palette…"); + assert_eq!(action_entry("ToggleSwitcher").1, "Switch Workspace…"); // The numbered families are templated, not nine strings per locale. assert_eq!(action_entry("ActivateTab3").1, "Go to Tab 3"); assert_eq!(action_entry("SelectWorkspace7").1, "Go to Workspace 7");