[앱 공통] 진입 화면 안전 영역 및 공지 모달 노출 순서 수정 - #121
Merged
Merged
Conversation
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
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.
작업 내용
앱 진입 화면과 초기화 화면에 기기 안전 영역을 반영했습니다.
공지 및 약관 모달의 노출 순서를 수정했습니다.
Android/iOS 안전 영역 및 전역 모달 노출 조건에 대한 회귀 테스트를 추가했습니다.
변경 이유
Android 실기기에서 하단 시스템 바가 진입 화면 좌표 계산에 반영되지 않아 브랜드 요소가 지나치게 아래쪽에 표시되는 문제를 수정하기 위함입니다.
또한 공지 모달이 네이티브 Modal로 진입 애니메이션보다 먼저 표시되면서, 모달 뒤에서 애니메이션이 끝나버리는 문제를 해결했습니다.
테스트한 내용
yarn test --runInBandyarn tsc --noEmityarn lintyarn bundle:androidyarn bundle:iosyarn build:android:debug스크린샷