-
-
Notifications
You must be signed in to change notification settings - Fork 650
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
Improve monochrome icon #2980
Improve monochrome icon #2980
Conversation
Hi Dubzer It seems you haven't yet signed a CLA. Please do so here. Once you do that we will be able to review and accept this pull request. Thanks! |
I'm not sure about the documentation. Is it required here? |
I agree it is good to better match other app's icon sizes, and it looks better in your screenshot, but it isn't working for me when building with the branch you supplied. The new drawables seem to be exactly the same as the existing foreground icon drawable? Also does
No, a change to the documentation wouldn't be required for this PR. |
I think the debug app icon should still be different than the production app icon, if we keep them the same it will be hard to distinguish between the 2 when themed icons are used on the device. |
I think they're currently also the same when using themed icons? It seems to be limited to the home screen, so not impossible to tell them apart (still using the default color in the app drawer, overview, ...). |
This was not intended. Fixed! Also, I removed debug version of the icon. |
Tested again and now the icon appears to match the screenshot. @dshokouhi Do you feel like there should be a different monochrome icon for the debug builds? Not sure how to interpret your 👍 on my last comment. |
My 👍 was in response to the icon already being used. I didn't realize that. Also you are correct that they will still be distinguishable there. |
Summary
I noticed that the Home Assistant monochrome app icon looked weird compared to the others. So, I re-generated it with Android Studio to better match the guideline grid.
Screenshots
Link to pull request in Documentation repository
Documentation: home-assistant/companion.home-assistant#
Any other notes