Skip to content

Add HomeSeer brand images#4250

Merged
frenck merged 1 commit into
home-assistant:masterfrom
b-uwe:homeseer_brand
Mar 28, 2023
Merged

Add HomeSeer brand images#4250
frenck merged 1 commit into
home-assistant:masterfrom
b-uwe:homeseer_brand

Conversation

@b-uwe
Copy link
Copy Markdown
Contributor

@b-uwe b-uwe commented Mar 28, 2023

Proposed change

Type of change

  • Add a new logo or icon for a new core integration
  • Add a missing icon or logo for an existing core integration
  • Add a new logo or icon for a custom integration (custom component)
  • Replace an existing icon or logo with a higher quality version
  • Removing an icon or logo

Additional information

none

Checklist

  • The added/replaced image(s) are PNG
  • Icon image size is 256x256px (icon.png)
  • hDPI icon image size is 512x512px for (icon@2x.png)
  • Logo image size has min 128px, but max 256px, on the shortest side (logo.png)
  • hDPI logo image size has min 256px, but max 512px, on the shortest side (logo@2x.png)

Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @b-uwe 👍

../Frenck

@frenck frenck merged commit 916459f into home-assistant:master Mar 28, 2023
@frenck frenck added the has-parent This PR has a parent PR in a other repo label Mar 28, 2023
Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare Mar 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recommend using a compression service on the images, eg https://tinypng.com/. I could compress this image by 62%.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch 🤦 ! I created #4252

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in a other repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants