Skip to content
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

Updated app icon #2414

Closed
wants to merge 1 commit into from
Closed

Conversation

declan-wade
Copy link

Summary

This to update the app icon to incorporate the new branding recently announced. Similar to a recently merged PR in the Android repo, refer here: home-assistant/android#3888. I note that the new icon when used on dark backgrounds is not white, it is a very light grey, as per the branding guidelines.

Refer to screenshot below to compare with the existing (icon 2 is the selected icon), below is an explanation of the options.
The image dimensions match the existing format.

Screenshots

screenshot

1 -- App icon matches almost exactly the android app. Seems too small, as Android icons (i.e. Pixel Launcher) are cropped to a circle, rather than iOS which just rounds the corners with a set radius.

2 -- App icon matches the existing iOS icon in size and position. This is not actually perfectly centered on the canvas, it is offset slightly on the Y-axis to be slightly higher and gives the illusion it is centered. This looks the most visually pleasing.

3 -- Current icon

4 -- App icon is perfectly centered and scaled to encompass the existing iOS icon, which results in it looking a bit too big and gives the illusion the icon sits slightly too low on the canvas.

Link to pull request in Documentation repository

Documentation: N/A

Aligns with new Home Assistant branding
@home-assistant
Copy link

Hi @declan-wade

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!

@bramkragten
Copy link
Member

See #2418

@robbiet480
Copy link
Member

Took #2418 instead of this one but thank you for your contribution @declan-wade!

@robbiet480 robbiet480 closed this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants