Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[리팩토링] 홈 화면 Status Bar 접근 로직 리팩토링 #320

Merged
merged 2 commits into from
Nov 30, 2021
Merged

[리팩토링] 홈 화면 Status Bar 접근 로직 리팩토링 #320

merged 2 commits into from
Nov 30, 2021

Conversation

sujeong000
Copy link
Member

작업 내용

  • Status Bar 접근 방식을 deprecated 되지 않은 방법으로 변경

시연 방법

기타 (고민과 해결, 리뷰 포인트 등)

  • UIApplication.shared.statusBar는 iOS 13.0 부터 deprecated 되었음 -> 방식 변경 필요!
  • StatusBar에 접근하려면 현재 view가 속한 window에 접근해야함
  • window를 가지고 있는 것은 AppCoordinator
  • 그리고 Home을 start하는 것도 AppCoordinator
  • 따라서 AppCoordinator에서 HomeCoordinator.start(statusBarHeight: CGFloat?) 로 전달해주도록 하였음.

@sujeong000 sujeong000 added 🧱 refactor 리팩토링, 기능적인 부분에 수정이 없이 전체적인 코드의 구조를 개선 👍 review 리뷰 요청시 사용하는 라벨 labels Nov 30, 2021
@Modyhoon Modyhoon merged commit ad7665e into develop Nov 30, 2021
@Modyhoon Modyhoon deleted the issue/#319 branch November 30, 2021 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmfrlrkvlek tmfrlrkvlek Awaiting requested review from tmfrlrkvlek

@Modyhoon Modyhoon Awaiting requested review from Modyhoon

Assignees

@minsangKang minsangKang

@sujeong000 sujeong000

Labels
🧱 refactor 리팩토링, 기능적인 부분에 수정이 없이 전체적인 코드의 구조를 개선 👍 review 리뷰 요청시 사용하는 라벨
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sujeong000 @minsangKang @Modyhoon