[ISSUE-32] 네비게이션 추가 #41
Merged
Conversation
This file contains 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
- 바텀 네비게이션 추가 - 화면에 따라 바텀 네비게이션 유무 다르게 적용
- make plan -> create plan
…navigation # Conflicts: # presentation/src/main/java/com/yapp/growth/presentation/MainActivity.kt
- Growth -> Planz
hoyahozz
reviewed
Jun 19, 2022
메인 홈에서 뒤로 가기 버튼을 누르면 이전 탭이 없기 때문에 종료됩니다(메인 홈이 가장 첫 화면이기 때문에..) 관리 탭에서 이전 버튼을 누르면 홈 화면으로 이동되고, 홈 화면에서 뒤로가기 버튼을 눌러야 종료되는 방식입니다 |
…navigation # Conflicts: # app/src/main/AndroidManifest.xml # presentation/src/main/res/values/strings.xml
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
ISSUE
작업 내용
p.s. 스플래시 쪽과 충돌이 많을 것으로 예상됩니다...
스플래시 화면에도 네비게이션이 적용되어야 할 것 같아서 스플래시 쪽 머지 완료되면 그때 충돌 잡아서 머지하겠습니다
현재 약속잡기 FAB 아이콘을 전달받지 못해서 디자이너분께 받으면 바로 수정할 예정입니다
남은 시간 동안은 자주 사용되는 컴포넌트들 추가해 놓을 예정입니다!
추가로 저희 상태바 투명하게 하는 부분 이슈도 추가되어야 할 것 같습니당..
실행 화면
Check List
[ISSUE-{N}] PR 제목으로 작성