-
Notifications
You must be signed in to change notification settings - Fork 531
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 part of #4177: Dark mode Topic Activity Part 2 (Info Tab) #4570
Fixes part of #4177: Dark mode Topic Activity Part 2 (Info Tab) #4570
Conversation
@rt4914 don't know why these test are failing. I only resolve confilct and update this branch as per Develop. It's buiding succesfull. Also In Test Fail it not specify any particular error. |
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.
@MohitGupta121 Suggested changes
@@ -4,6 +4,6 @@ | |||
<gradient | |||
android:angle="270" | |||
android:startColor="#00FFFFFF" |
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 create a similar value for startColor too
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 create a similar value for startColor too
Actually startColor not changing in dark_mode it's same. Still I have to make value?
@@ -246,5 +246,14 @@ | |||
<color name="component_color_confetti_red_color">@color/color_palette_confetti_red_color</color> | |||
<color name="component_color_confetti_yellow_color">@color/color_palette_confetti_yellow_color</color> | |||
<color name="component_color_confetti_blue_color">@color/color_palette_confetti_blue_color</color> | |||
<!-- Topic Info Activity --> |
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.
Change this to Topic Info Fragment
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.
Change this to
Topic Info Fragment
okay
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, Thanks.
Explanation
Fixes part of #4177 : Dark mode Topic Activity Part 2 (Info Tab)
Dark Mode Mockup:-
https://xd.adobe.com/view/c05e9343-60f6-4c11-84ac-c756b75b940f-950d/screen/3b067181-e5e0-4a5b-bbb8-ab5eda952549/specs/
Essential Checklist
For UI-specific PRs only
If your PR includes UI-related changes, then: