fix(ui): drop Ctrl+L hint from text selection menu
Keep the shortcut in Help/Settings only; the floating selection chip should stay quiet so the Add to Chat action is easier to scan. Co-authored-by: Serhii Dziupin <makeittech@users.noreply.github.com>
This commit is contained in:
co-authored by
Serhii Dziupin
parent
682c42df8f
commit
f4f23704b8
@@ -16,7 +16,6 @@ import { useEffectiveDirectory } from '@/hooks/useEffectiveDirectory';
|
|||||||
import { isVSCodeRuntime } from '@/lib/desktop';
|
import { isVSCodeRuntime } from '@/lib/desktop';
|
||||||
import { useI18n } from '@/lib/i18n';
|
import { useI18n } from '@/lib/i18n';
|
||||||
import { rangeToMarkdown, trimSelectionValue, wrapMarkdownSelectionForChat } from './selectionMarkdown';
|
import { rangeToMarkdown, trimSelectionValue, wrapMarkdownSelectionForChat } from './selectionMarkdown';
|
||||||
import { formatShortcutForDisplay, getEffectiveShortcutCombo } from '@/lib/shortcuts';
|
|
||||||
import { focusChatInput } from '@/components/chat/composer/editor/dom';
|
import { focusChatInput } from '@/components/chat/composer/editor/dom';
|
||||||
|
|
||||||
interface TextSelectionMenuProps {
|
interface TextSelectionMenuProps {
|
||||||
@@ -65,11 +64,6 @@ export const TextSelectionMenu: React.FC<TextSelectionMenuProps> = ({ containerR
|
|||||||
const currentSessionId = useSessionUIStore((state) => state.currentSessionId);
|
const currentSessionId = useSessionUIStore((state) => state.currentSessionId);
|
||||||
const setPendingInputText = useInputStore((state) => state.setPendingInputText);
|
const setPendingInputText = useInputStore((state) => state.setPendingInputText);
|
||||||
const isMobile = useUIStore((state) => state.isMobile);
|
const isMobile = useUIStore((state) => state.isMobile);
|
||||||
const shortcutOverrides = useUIStore((state) => state.shortcutOverrides);
|
|
||||||
const addToChatShortcut = React.useMemo(
|
|
||||||
() => formatShortcutForDisplay(getEffectiveShortcutCombo('add_selection_to_chat', shortcutOverrides)),
|
|
||||||
[shortcutOverrides],
|
|
||||||
);
|
|
||||||
const projects = useProjectsStore((state) => state.projects);
|
const projects = useProjectsStore((state) => state.projects);
|
||||||
const availableWorktreesByProject = useSessionUIStore((state) => state.availableWorktreesByProject);
|
const availableWorktreesByProject = useSessionUIStore((state) => state.availableWorktreesByProject);
|
||||||
const effectiveDirectory = useEffectiveDirectory();
|
const effectiveDirectory = useEffectiveDirectory();
|
||||||
@@ -425,9 +419,7 @@ export const TextSelectionMenu: React.FC<TextSelectionMenuProps> = ({ containerR
|
|||||||
'active:opacity-80',
|
'active:opacity-80',
|
||||||
'transition-opacity duration-150'
|
'transition-opacity duration-150'
|
||||||
)}
|
)}
|
||||||
title={addToChatShortcut
|
title={t('chat.textSelection.title.addToCurrentChat')}
|
||||||
? `${t('chat.textSelection.title.addToCurrentChat')} (${addToChatShortcut})`
|
|
||||||
: t('chat.textSelection.title.addToCurrentChat')}
|
|
||||||
type="button"
|
type="button"
|
||||||
>
|
>
|
||||||
<Icon name="add" className="h-5 w-5 flex-shrink-0" />
|
<Icon name="add" className="h-5 w-5 flex-shrink-0" />
|
||||||
@@ -520,16 +512,11 @@ export const TextSelectionMenu: React.FC<TextSelectionMenuProps> = ({ containerR
|
|||||||
'hover:bg-[var(--interactive-hover)]',
|
'hover:bg-[var(--interactive-hover)]',
|
||||||
'transition-colors duration-150'
|
'transition-colors duration-150'
|
||||||
)}
|
)}
|
||||||
title={addToChatShortcut
|
title={t('chat.textSelection.title.addToCurrentChat')}
|
||||||
? `${t('chat.textSelection.title.addToCurrentChat')} (${addToChatShortcut})`
|
|
||||||
: t('chat.textSelection.title.addToCurrentChat')}
|
|
||||||
type="button"
|
type="button"
|
||||||
>
|
>
|
||||||
<Icon name="add" className="h-4 w-4" />
|
<Icon name="add" className="h-4 w-4" />
|
||||||
<span className="whitespace-nowrap">{t('chat.textSelection.actions.addToChat')}</span>
|
<span className="whitespace-nowrap">{t('chat.textSelection.actions.addToChat')}</span>
|
||||||
{addToChatShortcut ? (
|
|
||||||
<span className="whitespace-nowrap text-xs text-muted-foreground">{addToChatShortcut}</span>
|
|
||||||
) : null}
|
|
||||||
</button>
|
</button>
|
||||||
|
|
||||||
<div className="w-px h-4 bg-[var(--interactive-border)]" />
|
<div className="w-px h-4 bg-[var(--interactive-border)]" />
|
||||||
|
|||||||
Reference in New Issue
Block a user