Fix pencil icon color to work both on light and dark mode #13316
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When beta testing 21.4, @itsmeichigo reported this issue where the pencil icon for editing status in Order Details is hard to see:
This PR sets the icon to use
color_primary
, which is commonly used in other tappable icons in the app. The result is like this:Additional discussion
The affected icon is actually used in different places in the app:
However, we're not hearing about color issue in dark mode in other places where this icon is used. This is because whenever it is used, either in Compose or XML, its tint is also set separately there (e.g: with
app:iconTint="@color/color_primary"
ortint = MaterialTheme.colors.primary
).With the change introduced in this PR, technically, those separate color settings are not needed anymore and can be deleted. However, deleting them means it will require us extra time to check each individual area that uses it. This will have to be done not just during PR review, but also later during beta testing.
Thinking about that trade-off, I decided not to remove the individual setting and just leave them be, to save us time.
Steps to reproduce
Build app, then edit an Order in Dark Mode. Ensure the pencil icon next to the Order status is visible in purple color, similar to screenshot above.
Testing information
The tests that have been performed
Listed above.
Images/gif
Added above.
RELEASE-NOTES.txt
if necessary. Use the "[Internal]" label for non-user-facing changes.Reviewer (or Author, in the case of optional code reviews):
Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement: