-
Notifications
You must be signed in to change notification settings - Fork 2
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
메인화면 식단 영역 최종 리뉴얼 및 QA 대응 #231
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다! 👍
} | ||
|
||
private fun updateMenu(list: List<Dining>, position: Int, arrangedList: List<Dining>) { | ||
val menus = listOf( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TextView가 많아서 관리하는 데 조금 어려움이 있지 않을까 생각이 들어요. 식단 상세 페이지의 경우 하나의 텍스트 뷰에서 줄바꿈을 통해 식단을 구분하여 표시하고 있는데, 그런 비슷한 느낌으로 텍스트 뷰를 줄이면 코드 작성이 더 수월해질 수 있을 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
} | ||
|
||
fun updateDining(list: List<Dining>, position: Int) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변수명을 list 대신 좀 더 명확한 것으로 사용한다면 다른 사람 입장에서 구분하기 좋을 것 같습니다!
이슈
작업사항
참고사항
구현 화면
(18시 01분 기준 - 에뮬레이터 직접 끈거에요 앱 터진거 아니에요)
2024-04-16.6.04.21.mov