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

[Setting] 라이트 모드 / 다크 모드 설정 카드 UI #61

Merged
merged 9 commits into from
Jul 22, 2023
Merged

[Setting] 라이트 모드 / 다크 모드 설정 카드 UI #61

merged 9 commits into from
Jul 22, 2023

Conversation

laco-dev
Copy link
Contributor

@laco-dev laco-dev commented Jul 20, 2023

Issue

Overview (Required)

  • 시스템 테마를 바꾸기 위해 AppCompat 기반의 구조로 테마 변경
  • 구성 변경이 발생할 때 탭 구조가 초기화 되지 않도록 NavController의 상태에 따라 현재 탭을 판단하도록 개선
  • 설정 화면의 다크 테마를 변경할 수 있는 카드 UI 구현

Issue

  • 다크모드를 강제로 바꾸면 화면이 새로 그려지면서 깜빡이는 이슈가 있음
  • 구성 변경을 수동으로 바꾸도록 해서 UI만 바뀌도록 대응한 상태
device-2023-07-20-235615.mp4

@laco-dev laco-dev self-assigned this Jul 20, 2023
@laco-dev laco-dev changed the title [Draft] [Setting] 라이트 모드 / 다크 모드 설정 카드 UI [Setting] 라이트 모드 / 다크 모드 설정 카드 UI Jul 20, 2023
@laco-dev laco-dev requested a review from wisemuji July 20, 2023 15:09
Copy link
Member

@wisemuji wisemuji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생 많으셨습니다 🙇 많은 고민의 흔적이 보이네요...

shape = RoundedCornerShape(32.dp),
color = Color(0xFFFFFFFF),
shadowElevation = 2.dp,
) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

카드 UI는 공통으로 관리하면 어떨까요? 꼭 이 PR이 아니여도 됩니다.

Copy link
Member

@wisemuji wisemuji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 👏

@laco-dev laco-dev merged commit 8995b33 into droidknights:main Jul 22, 2023
@laco-dev laco-dev deleted the feature/#54 branch July 22, 2023 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Setting] 라이트 모드 / 다크 모드 설정 카드 UI
3 participants