[BUG] Misleading hotkey conflict with browser incognito mode

[VooDisss]

Version 0.10.2

Parent issue (related): #152

Description

When I attempt to use the Ctrl+Shift+N (or Cmd+Shift+N) hotkey combination to create a new session, the web browser intercepts the command to open a new incognito/private window. This makes the hotkey misleading and effectively non-functional for the WebUI.

Expected Behavior

The application should use a unique hotkey combination that does not conflict with standard browser-level shortcuts, or it should rely on UI-based triggers if a conflict-free shortcut cannot be established.

Actual Behavior

Pressing the current combination triggers the browser's "New Incognito Window" action instead of the application's "Create Session" logic.

Technical Context

• File: packages/ui/src/components/instance-welcome-view.tsx
• Code Snippet:

  // Line 56
  const newSessionShortcutString = createMemo(() => (isMac() ? "cmd+shift+n" : "ctrl+shift+n"))
  
  // Line 112
  if ((e.metaKey || e.ctrlKey) && e.shiftKey && e.key.toLowerCase() === "n") {
    e.preventDefault()
    handleNewSession()
    return
  }

User Experience Goal

Users should have a reliable keyboard shortcut for session creation that behaves consistently across different browsers without triggering native browser features.

[shantur]

What shortcut do you think can be used?

[VooDisss]

@shantur I think that no hotkey is needed - in my opinion those shortcuts are to be removed. Also there are other shortcut combos that are not functional:

[VooDisss]

Issue fixed by #165 , waiting for @shantur approval and merging.

[shantur]

@CodeNomadBot

Understand how keyboard shortcut hints are implemented in packages/ui and then
check why keyboard shortcuts are not hiding when packages/ui is run on phone / tablet?

[VooDisss]

#165 rejected by @shantur .

I propose adding a "Toggle Shortcut Hints" feature. This allows power users to keep using hotkeys while providing a way to clean up the UI for those who find the visual hints distracting, especially on mobile.

Proposed Implementation Tracking:

1. Preference State: Add showKeyboardShortcuts: boolean (default: true) to the Preferences interface in packages/ui/src/stores/preferences.tsx.
2. Toggle Command: Register a new command in packages/ui/src/lib/hooks/use-commands.ts to flip this preference via the Command Palette.
3. Selective UI Hiding:
   • Sidebar: Wrap the shortcut containers in packages/ui/src/components/instance/shell/SessionSidebar.tsx (lines 122 and 169) with a check for this preference.
   • Welcome View: Wrap the .panel-footer-hints and the <Kbd> component inside the "Create Session" button in packages/ui/src/components/instance-welcome-view.tsx.
   • Global Hints: Apply the check to packages/ui/src/components/keyboard-hint.tsx to handle other hint rows systematically.

This preserves functionality for those who use it heavily while solving the visual clutter issue.

[codenomadbot]

Automation failed.

session.prompt failed: fetch failed

--
Yours, CodeNomadBot

[shantur]

@CodeNomadBot

First, understand how keyboard shortcuts are displayed in packages/ui, then

Raise a PR to fix this with 2 changes

1. Hide shortcuts in web mode similar to like we hide on phone and tablet.
2. Add a setting in Command Palette similar to other display toggles, something like "Show/Hide Keyboard Shortcut Hints". Choose relevant description.
3. When in web mode this setting will be disabled and web mode supersedes

[shantur]

@CodeNomadBot

Raise PR

[codenomadbot]

Automation failed.

session.command failed: fetch failed

--
Yours, CodeNomadBot

[shantur]

@CodeNomadBot

Try again

[codenomadbot]

Automation failed.

session.command failed: fetch failed

--
Yours, CodeNomadBot

[shantur]

@CodeNomadBot

Again