fix(nav): Cmd/Ctrl+K 단축키에도 isEditableTarget 가드 적용
리뷰어 지적: 주석엔 '다른 인풋/편집 영역에서 무시'라고 적어놨지만 실제 코드는 '/' 키에만 가드를 걸어둬서 Cmd/Ctrl+K 는 모든 input/textarea/contenteditable 에서 브라우저/편집기 단축키를 가로채는 상태였음. 수정: '/' 와 동일하게 !isEditableTarget(e.target) 조건 추가. 검색 인풋 자체에 포커스 중이면 어차피 focus()/setOpen(true) 가 noop 이라 UX 손해 없음.
This commit is contained in:
@@ -73,8 +73,13 @@ export function TopNav() {
|
||||
inputRef.current?.blur();
|
||||
return;
|
||||
}
|
||||
// Cmd/Ctrl+K — OS 공통 검색 단축키.
|
||||
if ((e.metaKey || e.ctrlKey) && e.key.toLowerCase() === "k") {
|
||||
// Cmd/Ctrl+K — OS 공통 검색 단축키. 단, 다른 인풋/편집 영역에서 입력 중이면
|
||||
// 브라우저/편집기 단축키를 뺏지 않도록 동일하게 무시.
|
||||
if (
|
||||
(e.metaKey || e.ctrlKey) &&
|
||||
e.key.toLowerCase() === "k" &&
|
||||
!isEditableTarget(e.target)
|
||||
) {
|
||||
e.preventDefault();
|
||||
inputRef.current?.focus();
|
||||
setOpen(true);
|
||||
|
||||
Reference in New Issue
Block a user