Skip to content

refactor: fadePosition enum을 픽셀 기반 페이드 설정으로 전환#56

Closed
eun-yeon wants to merge 1 commit intomasterfrom
fix/fade-migration-and-patch-completeness
Closed

refactor: fadePosition enum을 픽셀 기반 페이드 설정으로 전환#56
eun-yeon wants to merge 1 commit intomasterfrom
fix/fade-migration-and-patch-completeness

Conversation

@eun-yeon
Copy link
Member

@eun-yeon eun-yeon commented Mar 2, 2026

개요

페이드 위치를 드롭다운으로 고르던 방식에서, 상단/하단 페이드를 각각 픽셀 값으로 직접 설정하는 방식으로 변경
원래는 드롭다운에서 커스텀을 선택 시, 픽셀 값으로 변경할 수 있게 하려고 했지만 단순화를 위해 통일함

변경 내용

백엔드

  • fadeTopPx, fadeBottomPx, reverseFadeTopPx, reverseFadeBottomPx 필드 추가
  • 기존 fadePosition enum이 남아있는 store/프리셋은 자동으로 픽셀값으로 변환하는 마이그레이션 추가
  • 탭별 오버라이드도 동일하게 마이그레이션 처리
  • preset_load()keys_reset_all()에서 keyDisplayDelayMs가 빠져있던 기존 누락 수정

프론트엔드

  • 셰이더에서 enum 플래그 분기 로직 버리고, 픽셀값을 그대로 받아서 처리하도록 단순화
  • NoteSetting UI — 드롭다운 제거, 상단(↑)/하단(↓) 숫자 입력으로 교체. 리버스 모드일 때는 리버스용 값 편집 (별개)
  • WebGLTracksOGL의 useEffect 의존성 배열을 개별 필드로 분리해서 WebGLTracks랑 통일
  • 5개 언어 locale 정리

마이그레이션 매핑

fadePosition fadeTopPx fadeBottomPx reverseFadeTopPx reverseFadeBottomPx
auto 50 0 0 50
top 50 0 50 0
bottom 0 50 0 50
none 0 0 0 0
both 50 50 50 50

마이그레이션 끝나면 fadePosition은 auto로 리셋돼서 다시 돌지 않음

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lee-sihun lee-sihun closed this Mar 4, 2026
@lee-sihun lee-sihun deleted the fix/fade-migration-and-patch-completeness branch March 4, 2026 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants