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

[YDS-#214] isSystemInDarkTheme() 설정을 위한 YDSTheme에 매개변수 추가 #267

Merged
merged 2 commits into from
Mar 24, 2024

Conversation

kangyuri1114
Copy link
Member

Summary

  • isSystemInDarkTheme() 설정을 위헤 YDSTheme에 매개변수를 추가해 다크모드 여부 인자를 외부에서 받을 수 있습니다
  • versionName 2.5.9 -> 2.5.10

Describe your changes

YdsTheme(isDarkMode = true) { ... }

다크모드 시 위의 코드처럼 사용하면 될 것 같습니다!

Issue

To reviewers

제가 이슈를 잘 이해한 건지 확신은 안 드는데..
궁금해서 그런데 외부 인자로 받도록 수정해야 했던 이유가 무엇인지 알 수 있을까요?!

PR 올리기 전 체크 리스트

  • version.properties의 버전을 업데이트 했나요?

@kangyuri1114 kangyuri1114 requested a review from a team March 24, 2024 08:53
@kangyuri1114 kangyuri1114 self-assigned this Mar 24, 2024
Copy link
Member

@cometj03 cometj03 left a comment

Choose a reason for hiding this comment

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

이슈 잘 이해해주신 거 맞아요!

이 이슈를 만든 이유는 예를 들어 시스템 다크모드 여부와 관계없이 라이트모드로 설정하고 싶을 때가 있는데, 이전 버전에서는 그렇게 할 수 있는 방법이 없었어요

그래서 사용하는 곳에서도 임의로 다크모드 여부를 설정할 수 있게 하기 위해 해당 이슈를 올렸습니다!

Copy link
Member

@giovannijunseokim giovannijunseokim left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 점점 기여량이 늘어나시는군요 ㅎㅎ 👍👍👍

@kangyuri1114 kangyuri1114 merged commit 44f8035 into develop Mar 24, 2024
1 check passed
@kangyuri1114 kangyuri1114 deleted the feature/rein/add_darkmode_parameter branch March 24, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

YdsTheme isDarkMode 인자 추가
3 participants