-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fix icon padding #95
Comments
Hi Mikhail, thanks for the feedback. |
I have to take a look at the adaptive icon implementation, maybe there's something wrong... |
@mchowdhury Does that look as intended? |
@cyb3rko Looks great! Can you check that the the shortcut icons (Minimum, Maximum Brightness, etc.) have the correct padding too? |
Perfect, I'll try my best |
Great app, I love using it!
There is a minor issue where the padding around logo on the main icon as well of the shortcut actions don't match the Android guidelines. The guidelines recommend 18dp of padding around the icon and the edge of the circle.
See the screenshot below where the all the other icons respect the 18dp padding rule and flashdim icon has almost no padding.
The text was updated successfully, but these errors were encountered: