Task #223: Ctrl/Cmd+O 파일 열기 단축키 추가#749
Closed
oksure wants to merge 2 commits into
Closed
Conversation
shortcut-map에 Ctrl+O → file:open 매핑 추가. setupGlobalShortcuts에 Ctrl/Cmd+O 핸들러 추가 — 문서 미로드 상태에서도 동작. 한글 IME 상태(ㅐ)도 처리. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Ctrl+O(Windows) / Cmd+O(macOS)로 “파일 열기”를 지원하도록 단축키 체계를 확장해, 문서 미로드 상태에서도 기본 편집기 수준의 열기 UX를 제공하려는 PR입니다.
Changes:
setupGlobalShortcuts()에 Ctrl/Cmd+O 전역 처리 추가(문서 미로드 상태에서도file:open디스패치)- 기본 단축키 매핑에 Ctrl+O →
file:open추가
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| rhwp-studio/src/main.ts | 문서 미로드 상태에서 Ctrl/Cmd+O를 가로채 file:open을 실행하도록 전역 키다운 핸들링 추가 |
| rhwp-studio/src/command/shortcut-map.ts | 편집기 단축키 테이블에 Ctrl+O → file:open 매핑 추가 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
19
to
23
| // 파일 | ||
| [{ key: 'n', alt: true }, 'file:new-doc'], | ||
| [{ key: 'ㅜ', alt: true }, 'file:new-doc'], | ||
| [{ key: 'o', ctrl: true }, 'file:open'], | ||
| [{ key: 's', ctrl: true }, 'file:save'], |
한글 입력 상태에서 'O' 키는 'ㅐ'로 입력됨. 기존 Alt+N ↔ ㅜ 패턴처럼 Ctrl+O ↔ ㅐ 매핑을 shortcut-map에 추가. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Copilot 리뷰 반영: a246d962에서 한글 IME 상태(ㅐ)에서도 Ctrl+O가 동작하도록 shortcut-map에 매핑 추가했습니다. |
edwardkim
added a commit
that referenced
this pull request
May 10, 2026
기존 Alt+N (새 문서) 패턴 정합 — setupGlobalShortcuts + shortcut-map 영역 영역 동일 패턴 적용: - Ctrl/Cmd+O → file:open (문서 미로드 상태에서도 동작) - 한글 IME 매핑: Ctrl+ㅐ (Alt+ㅜ 패턴 정합) 검증: - tsc --noEmit ✅ - cargo test --release ALL GREEN - 광범위 sweep 170/170 same - WASM 4.66 MB - 웹 에디터 인터랙션 검증 ✅ 통과 Issue #223 OPEN 유지 — Cmd+Arrow / Opt+Arrow 등 후속 단계 잔존 (PR #746 의 Ctrl/Cmd+Arrow 영역 별도 단계)
Owner
|
@oksure 검토 완료했습니다. 감사합니다. 처리 결과Merge commit: 본질기존 Alt+N (새 문서) 패턴 정합 — 인프라 재사용
검증
Issue #223 은 OPEN 유지합니다 — Cmd+Arrow / Opt+Arrow / 한컴 주요 단축키 영역 의 후속 단계 잔존. 5/10 사이클 17번째 PR — 수고하셨습니다. |
Owner
|
Merged. Merge commit: 55cc65d |
edwardkim
added a commit
that referenced
this pull request
May 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
Ctrl+O (Windows) / Cmd+O (macOS) 단축키로 파일 열기가 지원되지 않습니다. 대부분의 문서 편집기에서 기본 제공하는 단축키입니다.
수정 내용
shortcut-map.ts:Ctrl+O → file:open매핑 추가main.tssetupGlobalShortcuts: 문서 미로드 상태에서도Ctrl/Cmd+O로 파일 열기 대화상자 호출ㅐ)도 처리범위
이 PR은 #223의 일부로, 파일 열기 단축키만 구현합니다.
검증
감사합니다.