-
Notifications
You must be signed in to change notification settings - Fork 499
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
Fixes #4182: Added dark mode support to OptionActivity and OptionWithoutDrawerActivity #4284
Fixes #4182: Added dark mode support to OptionActivity and OptionWithoutDrawerActivity #4284
Conversation
@rt4914 @ayush0402 PTAL, thanks. |
Hi @Akshatkamboj14, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Akshatkamboj14 Suggested changes
… Add-dark-mode-to-OptionActivity-and-OptionWithoutDrawerActivity
… Add-dark-mode-to-OptionActivity-and-OptionWithoutDrawerActivity
…_cyan.xml and updated related files
@rt4914 @ayush0402 @BenHenning PTAL. |
@ayush0402 can you please take a first pass? I'm otherwise assuming @rt4914 will follow up as the code owner, but please let me know if I need to review anything here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one change in PR description is required.
<stroke | ||
android:width="1dp" | ||
android:color="@color/color_def_black_12" /> | ||
</shape> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please mention the in the PR description why this and the other file has been introduced.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay @rt4914
Assigning @BenHenning for code owner reviews. Thanks! |
…nWithoutDrawerActivity
Hi. Since I was requested as a reviewer on this PR: Please note that I will be away & unavailable for reviews starting today until 23 May 2022. Please try to make any progress that you can which doesn't require me, and I'll catch up on my reviews when I return. Thanks for your flexibility--I really appreciate it. |
Hi @Akshatkamboj14, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue. |
Explanation
Fixes #4182
Added dark mode support to OptionActivity and OptionWithoutDrawerActivity
I have introduced two drawable resource files general_item_border.xml and general_item_border_cyan.xml as the original drawable resource files general_item_background_border.xml and general_item_background_border_cyan.xml are getting used in 7 different places and to have a consistent UI throughout the application so separate drawable resource files should be preferable.
Essential Checklist
For UI-specific PRs only
If your PR includes UI-related changes, then: