Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] #400 MetronomeView UI 업데이트 #407

Merged
merged 9 commits into from
Mar 26, 2025

Conversation

YunKi-H
Copy link
Collaborator

@YunKi-H YunKi-H commented Mar 22, 2025

개요

MetronomeView 관련 UI를 업데이트 합니다

작업 내용

1. MetronomeSettingControlView

  • 버튼의 모양이 중복되는거같아 ButtonStyle로 분리했습니다.
  • 사운드 변경 버튼의 경우 Menu 는 isPressed의 작동 로직이 약간 달라 따로 만들었습니다.
  • Menu 버튼에서 악기를 순차적으로 불러오기 위해
    • Instrument 열거값에 .나무 추가했습니다.
    • Instrument 열거값에 caseIterable 프로토콜 채택했습니다.

2. 사운드 변경 기능

  • AppState의 isBeepSound가 필요 없어짐에 따라
  • SoundManager에서 isBeepSound에 따른 로직 제거했습니다
  • 그냥 isBeepSound 필요 없어져서 다 지움
  • 삭제 이후 혹시나 동일한 이름으로 UserDefault 만들까봐 실행시 "isBeepSound" 키값 삭제 로직 추가

3. 메뉴버튼

  • 장단 연습 화면의 우측상단 메뉴버튼 수정
  • 커스텀 연습 화면에서 비프음으로 전환 메뉴 삭제
  • 빌트인 연습 화면에서 비프음으로 전환 메뉴 삭제
  • 빌트인 연습 화면 메뉴버튼 아이콘 변경 -> square.and.arrow.up

4. 기타 수정

  • 세팅컨트롤뷰에서 ViewModel 외부에서 주입받도록 변경
  • BuiltInJangdanPracticeView 에서 변수 사용할때 self. 붙임
  • BuiltInJangdanPracticeView 에서 Alert 내용 수정
  • CustomJangdanPracticeView 에서 Alert 텍스트 수정

비고

작업 스크린샷

리뷰어에게 남길 말

CheckList

  • PR의 Target이 올바르게 설정되어 있나요?
  • Assignee는 올바르게 할당되어있나요?
  • Label이 적절하게 설정되어 있나요?
  • 변경사항에 의문이 드는 곳은 없나요?

@YunKi-H YunKi-H added the enhancement New feature or request label Mar 22, 2025
@YunKi-H YunKi-H added this to the 2.5.0 milestone Mar 22, 2025
@YunKi-H YunKi-H self-assigned this Mar 22, 2025
@YunKi-H YunKi-H requested a review from l1004ga as a code owner March 22, 2025 11:43

if toastAction {
Text("'\(inputCustomJangdanName)' 내보내기가 완료되었습니다.")
if self.toastAction {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

self다 넣어주셨군요! 감사합니닷!

@l1004ga
Copy link
Collaborator

l1004ga commented Mar 26, 2025

[오류 재현 상황]
장구로 최초 선택하고 들어감
진양으로 테스트 했구요 소리를 장구, 북, 나무 전환 계속 하다보면 한번도 안바뀌다가 계속 바꿔보면 어느순간 소리가 바뀌면서 동시에 UI가 먹통이 됩니닷….

  1. 소리 전환이 정상적으로 안됨
  2. 소리가 갑자기 전환되었는데, UI가 멈췄습니닷...

정말 신기한건 멈춘 상태에서 백그라운드에서 다른 앱 갔다가 돌아오면 자진모리가 되어있어요... 하지만 계속 멈춰있는 자진모리ㅠㅠ

@YunKi-H
Copy link
Collaborator Author

YunKi-H commented Mar 26, 2025

[오류 재현 상황] 장구로 최초 선택하고 들어감 진양으로 테스트 했구요 소리를 장구, 북, 나무 전환 계속 하다보면 한번도 안바뀌다가 계속 바꿔보면 어느순간 소리가 바뀌면서 동시에 UI가 먹통이 됩니닷….

  1. 소리 전환이 정상적으로 안됨
  2. 소리가 갑자기 전환되었는데, UI가 멈췄습니닷...

정말 신기한건 멈춘 상태에서 백그라운드에서 다른 앱 갔다가 돌아오면 자진모리가 되어있어요... 하지만 계속 멈춰있는 자진모리ㅠㅠ

커밋 - 008ed69
에서 수정했습니다..
UI 프리징에 대해서는 짐작가는게 없네요

appState의 악기 선택만 실행하고 해당 악기를 반영하진 않아버린

@l1004ga
Copy link
Collaborator

l1004ga commented Mar 26, 2025

커밋 - 008ed69 에서 수정했습니다.. UI 프리징에 대해서는 짐작가는게 없네요

appState의 악기 선택만 실행하고 해당 악기를 반영하진 않아버린

이제 잘 바뀌긴 하는데,, 바꾸다보면 또 먹통이 됩니다... UI가 멈췄어요...

@YunKi-H
Copy link
Collaborator Author

YunKi-H commented Mar 26, 2025

커밋 - 008ed69 에서 수정했습니다.. UI 프리징에 대해서는 짐작가는게 없네요
appState의 악기 선택만 실행하고 해당 악기를 반영하진 않아버린

이제 잘 바뀌긴 하는데,, 바꾸다보면 또 먹통이 됩니다... UI가 멈췄어요...

#409
해당 이슈에서 해결해보겠습니다..

@YunKi-H YunKi-H merged commit ff00899 into dev Mar 26, 2025
3 checks passed
@YunKi-H YunKi-H deleted the feat/#400-Update-Metronomeview-UI branch March 26, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants