Merged
Conversation
중요 알림이거나 관리하지 않는 아이콘의 경우 적절한 아이콘이 표시되도록 함. 이를 위해 새로운 아이콘 추가.
설정 페이지에서 이동하는 또 다른 페이지들의 TopBar 아래에 구분선 추가 앱, 제목 페이지에서 중요/비중요 알림을 나누는 구분선의 경우 중요 알림이 있을 경우에만 표시되도록 개선
onComplete로 로딩 함수 넣어줌. 그리고 title view에 주석 추가
마지막 아이템이 삭제되었음에도 해당 아이템이 사라지지 않고 UI에 남아있던 현상 수정
공통된 것끼리 묶고, 패키지로 분리하여 탐색과 이해가 쉽도록 함.
모달창에 공통적으로 활용되는 Box 컴포넌트들을 각각 하나의 Composable로 묶어 관리하도록 함. 이를 통해 modal 상자 변화를 중앙 관리할 수 있게 되었음.
모달창 각 행동에 맞는 아이콘을 텍스트 왼쪽에 놓아둠.
패딩을 줄이니 너무 답답해 보여서 원래대로 수정.
Closed
5 tasks
Code Coverage
|
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
PR
작업 내용
참고 사항
'제목 화면에서 빠르게 뒤로 가기 -> 설정을 누르면 흰 화면이 뜨는 현상'의 경우
상태 관리 문제 같은데, 어떤 식으로 해도 똑같은 버그가 있습니다.
이후 설정 페이지 이동을 스와이프 방식으로 바꿀 것이므로, 그때에 다시 시도해겠습니다.
위 내용과 함께 UI 개선을 마무리하면 프로덕션 준비를 해도 괜찮을 것 같습니다.
✏ Git Close
#54