feat: 사운드·폰트 피커 인라인 관리#92
Open
eun-yeon wants to merge 2 commits into
Open
Conversation
- 피커 + 버튼이 파일 선택 → 트림 모달로 바로 이어지도록 변경 - 행 우클릭/호버 ⋮ 메뉴로 편집·삭제 처리 (내장 사운드는 메뉴 미노출) - 삭제 전 danger 확인 다이얼로그 추가 - 재생과 무관하던 사운드 라이브러리 enabled 토글 제거 (UI 체크박스, sound_set_enabled 커맨드, dmn.sound.setEnabled API) - SoundManagerModal 삭제, 피커 행 액션용 usePickerItemMenu 훅 추가
- FontManagerModal 제거, + 버튼이 로컬/웹폰트 추가 메뉴로 바로 연결 - 행 우클릭/⋮ 메뉴로 편집(웹)·이름 변경·활성화·삭제 처리 - 비활성 폰트를 숨기지 않고 흐리게 표시, 복원 실패 폰트는 재활성화 차단 - 폰트 라이브러리 액션을 useFontLibrary 훅으로 이전, 저장 실패 시 settings.get으로 백엔드 실상태 재조정 - 사운드/폰트 이름 변경 추가 (sound_rename 커맨드 + 인라인 rename) - ⋮ 재클릭 시 메뉴가 닫혔다 다시 열리는 깜빡임 수정 (outside-click pointerdown 레이스, pointerdown 스냅샷 토글로 해결) - 행 keydown 가드로 키보드에서 ⋮ 메뉴 열기 복구, ⋮ hover 배경 제거
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
1 participant
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.
개요
사운드와 폰트 피커에서 별도 매니저 모달 없이 목록에서 바로 추가·삭제·이름변경할 수 있게 개편
변경 내용
스택 1/15 · base
master순서대로 merge commit으로 머지 필요, squash나 rebase로 머지하면 위 PR들이 깨짐