Feature/nexsong with mini player #150
Merged
Conversation
This file contains 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
- 현재 재생중인 곡 UI 추가 - 드래그 시 이미지 수정 - 플레이리스트 터치영역 수정 - 플레이리스트 순서 초기값 추가
- PlayList -> Playlist - coredata 정렬 방식 변경
- player 변경 시 .replaceCurrentItem() 이용하도록 수정 - player 옵셔널 제거 - audioManager에서 progressbar 관련 변수, 함수 제거
- selectedTeam -> song.team으로 변경 - 미디어 플레이어 설정 복원
- 최신곡이 맨 마지막으로 - move 함수 수정
- 맨 마지막곡 재생 중 위로 한칸 올릴 경우 index 변화 감지 안되는 오류 존재
- 추후 메인화면으로 나가는 동작 추가 필요
- 추후 애니메이션 연결 필요
- isPlaying값에 맞춰서 애니메이션 재생 및 정지 - toast message 글자 색상 변경
- introspect 패키지로 해결 - 재생바 안에서 AudioManager 직접선언으로 player전달방식 수정
- currentIndex -> song.id (songDetailView) - currentSongId 삭제
-전체 재생 버튼 UI추가
- 음악 재생 변경 트리거 song -> currentSongId로 변경하여 여러 뷰에서 접근가능하도록 함
- SongDetailView / SongDetailViewModel 삭제 - 불필요한 주석 삭제
- 메인화면으로 나가기 - 미니플레이어 동작 제한 - 미니플레이어 버튼 색상 변경 - 불필요한 프린트문 제거
- 미니플레이어 좌우, 하단 여백 추가 - 메인 전체 재생하기 위아래 여백 수정 - 불필요한 스택 및 ignore 제거
- 선수 응원가 터치 영역 수정
- 한곡 인 경우 다음곡 이전곡 버튼 비활성화 (미니플레이어, 재생화면, 잠금화면) - 재생바 업데이트 시 미디어 플레이어와 연동 - 불필요한 프린트문 제거
- button -> onTapGesture로 변경
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.
Issue
Key Changes