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

[iOS] 여정 조회 개선 #350

Merged
merged 19 commits into from
Jan 26, 2024
Merged

[iOS] 여정 조회 개선 #350

merged 19 commits into from
Jan 26, 2024

Conversation

SwiftyJunnos
Copy link
Member

@SwiftyJunnos SwiftyJunnos commented Jan 12, 2024

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • 여정 조회 화면 개선

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

push ➡️ present

이전에 push를 사용해 UIPanGestureRecognizer를 사용해 여정 조회를 종료시킬 때
화면이 옆으로 이동한 빈 공간이 검은 색인 문제를 해결하고자
화면 전환 방식을 push에서 present로 변경하고 presentationStyle을 overFullscreen으로 설정했습니다.

그 과정에서 presentationStyle에 왼쪽 모서리 ➡️ 오른쪽으로 전환하는 스타일이 없어
위 ➡️ 아래로 내려야 화면이 종료되도록 수정해주었습니다.

데모

MSMusicPlayerView & .stopped 상태 시 화면 종료

별도의 컴포넌트로서 MSUIKit에 분리했습니다.

추가적으로 Bool 프로퍼티 isPlaying으로 재생 여부를 관리하던 방식에서
PlaybackStatus 열거형을 사용하는 방식으로 개선해주었습니다.\

.stopped 상태에서는 여정 조회 화면이 dismiss되도록 수정해주었는데,
이를 위해 .stopped 상태와 .paused 상태의 구분이 필요했기 때문입니다.

📸 스크린샷

화면 진입 화면 종료

@SwiftyJunnos SwiftyJunnos self-assigned this Jan 12, 2024
@SwiftyJunnos SwiftyJunnos linked an issue Jan 12, 2024 that may be closed by this pull request
3 tasks
@SwiftyJunnos SwiftyJunnos added the ♻️ 리팩토링 코드 동작 방식 변경, 리팩토링 label Jan 12, 2024
@SwiftyJunnos SwiftyJunnos marked this pull request as ready for review January 13, 2024 03:51
@SwiftyJunnos SwiftyJunnos added this to the 🌠 여정 조회 milestone Jan 13, 2024
@SwiftyJunnos SwiftyJunnos merged commit 8ac1187 into iOS/release Jan 26, 2024
31 checks passed
@SwiftyJunnos SwiftyJunnos deleted the iOS/task/Rewind-Fix branch January 26, 2024 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩토링 코드 동작 방식 변경, 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

여정 조회 개선
1 participant