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.

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

Make ThemeProvider singleton using Hilt #24

Closed
wants to merge 3 commits into from
Closed

Make ThemeProvider singleton using Hilt #24

wants to merge 3 commits into from

Conversation

7loro
Copy link
Owner

@7loro 7loro commented Oct 5, 2021

  • Hilt 를 이용하여 ThemeProvider 싱글턴 제공
  • Settings 모듈에서 Shared 모듈의 리소스를 직접 참조하는 대신 상수값 참조하도록 변경
  • theme 설정 시점 Activity->Application
      - Activity.onCreate 에서 수행 시 uiMode 값 변경에 따라 Activity 새로 생성 됨
      - Application 에 해당 로직을 넣어 Activity 두 번 생성되지 않게 적용 가능

TODO: 다크 모드로 앱 설정, 시스템은 light 일 때 앱 종료 상태에서 진입 시 white 윈도우 보여졌다가 뒤늦게 바뀌는 이슈

Giung Nam added 2 commits October 6, 2021 00:20
- ThemeProvider 싱글턴
- Settings 모듈에서 Shared 모듈의 리소스를 직접 참조하는 대신 상수값 참조하도록 변경
- Hilt 를 이용하여 ThemeProvider 싱글턴 제공
- theme 설정 시점 Activity->Application
  - Activity.onCreate 에서 수행 시 uiMode 값 변경에 따라 Activity 새로 생성 됨
  - Application 에 해당 로직을 넣어 Activity 두 번 생성되지 않게 적용 가능

TODO: 다크 모드로 앱 설정 해놓고 시스템은 light 일 때 앱 종료 상태에서 진입 할 때 white 윈도우 보여졌다가 뒤늦게 바뀌는 이슈
@7loro 7loro added enhancement New feature or request dependency library dependency management labels Oct 5, 2021
@7loro 7loro added this to the 1.0 milestone Oct 5, 2021
@7loro 7loro self-assigned this Oct 5, 2021
@7loro 7loro requested a review from amoseui as a code owner October 5, 2021 16:28
@7loro
Copy link
Owner Author

7loro commented Oct 5, 2021

#25
위 PR 과 합쳐져서 본 PR은 제거합니다

@7loro 7loro closed this Oct 5, 2021
@7loro 7loro deleted the refactoring branch October 6, 2021 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency library dependency management enhancement New feature or request
Projects
1st Iteration
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

1 participant