[refactor/#85] 부모 일정 뷰 수정 및 전체 구조, 화면 변경#94
Conversation
There was a problem hiding this comment.
고생많으셨습니다! 덕분에 편하게 작업할 것 같습니다 ㅎㅎ
대신 @Preview안에 screen에 theme 지정 부탁드려요!! 안그러면 preview 시 providers 에러 발생할 것 같습니다~
기존 스케쥴 관리뷰에서 일정/미션으로 나뉘는 구조였는데, 각 화면들이 명확히 분리
됨에 따라 feature 별로 패키징 수정했습니다.
라고 하셨는데 상위의 navigation과 component와 feature가 겹치는 구조라 screen이나 ui 등으로 패키징을 하나 더 해서 상위에서는 패키지 3개(navigation, component, ui 또는 feature로) 해보는 것이 어떨까요?
| iconRes = R.drawable.ic_parent_tab_today, | ||
| contentDescription = R.string.parent_journey_tab_content_description, | ||
| labelRes = R.string.parent_journey_tab_content_description, | ||
| route = ParentJourney, |
There was a problem hiding this comment.
현재 parent의 route들에 어떤 것은 parent 가 붙고 어떤 것은 안 붙어져 있는 상태이고 kid 역시 그런데 이것을 동일하게 parent나 kid를 붙이도록 하는게 일관성이 있어 파악하기 쉬울까요? 아니면 괜찮을까요?
고민 중인데 여러분 생각은 어떠신가요?
@Team-Kiero/kiero-android
There was a problem hiding this comment.
parent와 kid route 들 중에 중복되는 것들을 구분하기 위해 1차적으로 중복되는 것들에만 Parent 를 붙혔는데, 일관성 있게 처리하는게 좀 더 파악하기 쉬워보이긴 하네요!! 다른 분들 의견 들어보고 일치하면 route들 앞에 parent/kid 로 구분해 놓겠습니다! 감사합니당
There was a problem hiding this comment.
저두 일관성있는게 앞에 붙이는게 나중에 헷갈리지 않고 좋을 것 같다는 생각입니다!
...main/java/com/kiero/presentation/parent/mission/component/datepicker/model/DownTimeStatus.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/kiero/presentation/parent/mypage/ParentMypageScreen.kt
Outdated
Show resolved
Hide resolved
이거 변경되었네요.. ㅠㅠ 반영 부탁드려용
|
럴수 럴수 이럴수가... 변경사항 적용했습니다~ 감사합니당 |
dmp100
left a comment
dmp100
left a comment
There was a problem hiding this comment.
전체적인 패키지 구조 정리 및 화면 세팅 수고하셨습니다! 덕분에 작업 편하게 할 수 있을 것 같습니다. 풀 받게되면 mission 안에 자동추가 미션뷰랑 직접 입력뷰 두개로 나눠볼께용~
| iconRes = R.drawable.ic_parent_tab_today, | ||
| contentDescription = R.string.parent_journey_tab_content_description, | ||
| labelRes = R.string.parent_journey_tab_content_description, | ||
| route = ParentJourney, |
There was a problem hiding this comment.
저두 일관성있는게 앞에 붙이는게 나중에 헷갈리지 않고 좋을 것 같다는 생각입니다!
5 participants
ISSUE
❗ WORK DESCRIPTION
edit.1 -> preview theme 적용
edit.2 -> 부모 일정 empty view 변경사항 적용
edit.3 -> parent/component,navigation,screen/feature
edit.4 -> dummyfile 제거, 앱 메인로고 변경
📢 TO REVIEWERS
(담당 작업자 분들 편하시라고 일괄적으로 작업을 해놓긴 했는데, 오늘의 현황 글자를 잘못봐서 그냥 똑같이 Journey로 해버려서 앞에 parent/kid 로 분리했습니다.. 키하하...)
📸 SCREENSHOT