노래 재생 화면에서 플레이 리스트에 추가 #296
Conversation
| @Serializable | ||
| data class AddMusicToPlaylistRequest( | ||
| val musicId: String | ||
| ) |
There was a problem hiding this comment.
오 이거 제가 올린 PR에도 생성된 requestbody 인데 자주 쓰일 것 같아서 저는 MusicIdRequest 로 클래스를 만들었습니당! 이따 통일하는 게 좋을 것 같네요~!
| private fun List<Playlist>.toUiModels(): List<PlaylistUiModel> = this.map { it.toUiModel() } | ||
|
|
||
| private fun Playlist.toUiModel(): PlaylistUiModel { |
There was a problem hiding this comment.
함수명을 toPresentation 으로 하는 것 어떨까욥? toDomain 과 맥락이 동일하게!
There was a problem hiding this comment.
나는 조금 다르게 생각했어. UiModel은 특정 UI 컴포넌트에서만 사용하는 개념으로 봤어.
만약 presentation이라고 짓는 거라면 UI의 형태와 관계 없이 RecyclerView든 어떤 자세히보기 화면이든 관계 없이 사용할 수 있어야 할 것 같아. 이 경우엔 특정 RecyclerView에서만 사용할 수 있는 data class라고 생각해서 presentation에서 사용하는 model보다는 특정 ui에서 사용하는 model이다 라는 의미로 사용했어
There was a problem hiding this comment.
toPresentation == toUiModel 은 같은 의미라고 느껴져! 적어둔 이유처럼 특정 ui컴포넌트만을 위한 것이라면 어떤 컴포넌트의 UiModel인지 함수명을 바꿔야하지 않을까? 그게 아니라면 toPresentation 도 괜찮은 것 같아
| @@ -34,7 +34,7 @@ | |||
| <string name="description">설명</string> | |||
| <string name="select_thumbnail">썸네일 이미지 선택</string> | |||
|
|
|||
|
|
|||
| <string name="track_size">트랙 %d개</string> | |||
There was a problem hiding this comment.
이거 playlist 모듈 strings 파일에 있는데 playlist 모듈에서 삭제한번 해주세요!~
| import kotlinx.coroutines.launch | ||
|
|
||
| @AndroidEntryPoint | ||
| class PlaylistBottomSheet : BottomSheetDialogFragment() { |
There was a problem hiding this comment.
BottomSheetDialogFragment() 을 사용해서, Bottom Player가 안보이는 등, 적어주셨던 것 처럼 UI에 문제가 생기는 것거 같은데,
Fragment를 띄우면 하단 재생 바가 자기 부른 줄 알고 같이 올라옴
이유는 모르겠으나 바텀 시트가 떠 있는 동안 재생 버튼이 사라짐
UI radius 추가하거나 디자인을 바꿔야 함
최종발표까지 시간이 얼마 안남았기에, BottomSheetDialog 나 아니면 단순하게 Dialog 등으로 하는 것도 하나의 방법일 것 같아요.
(UI 문제가 빠르게 해결되면, 지금 상태로 해도 당연히 좋습니다.)
There was a problem hiding this comment.
Fragment가 아닌 클래스가 있었군요
그런데 지금은 Navigation과 Fragment의 기능을 사용하고 있기 때문에, 변경해야 할 지점이 많아요. 일단 이슈를 만들어 놓고 머지할게요. 모달 바텀 시트의 배경이라 기능상 문제가 있거나 크게 신경쓰이는 부분은 아닙니다!
Issue
Overview
Screenshot
To Reviewers
아래는 고쳐야 할 내용 (이슈 생성해야 함)