Skip to content

Commit 239ef1c

Browse files
fix(web): show the correctly matching shortcut in new thread button's tooltip (pingdotgg#5594)
1 parent 61b51ae commit 239ef1c

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

‎apps/web/src/components/SidebarV2.tsx‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2816,11 +2816,13 @@ export default function SidebarV2() {
28162816
openCommandPalette({ open: "new-thread-in" });
28172817
}, [isMobile, newThreadContext, projectGroups.length, setOpenMobile]);
28182818

2819-
// Same resolution as v1: prefer the local-thread binding, fall back to
2820-
// chat.new, no platform gating — web users have working shortcuts too.
2819+
// The button mirrors chat.new: in multi-project setups both route through
2820+
// the command palette's "New thread in..." picker, and in single-project
2821+
// setups both create immediately. chat.newLocal always creates directly, so
2822+
// it is only a correct label when chat.new is unbound.
28212823
const newThreadShortcutLabel =
2822-
shortcutLabelForCommand(keybindings, "chat.newLocal") ??
2823-
shortcutLabelForCommand(keybindings, "chat.new");
2824+
shortcutLabelForCommand(keybindings, "chat.new") ??
2825+
shortcutLabelForCommand(keybindings, "chat.newLocal");
28242826
return (
28252827
<>
28262828
<SidebarChromeHeader isElectron={isElectron} />

0 commit comments

Comments
 (0)