Skip to content

Add Icons to Discovered Integrations#5502

Closed
timmo001 wants to merge 3 commits intohome-assistant:devfrom
timmo001:integrations-icons
Closed

Add Icons to Discovered Integrations#5502
timmo001 wants to merge 3 commits intohome-assistant:devfrom
timmo001:integrations-icons

Conversation

@timmo001
Copy link
Copy Markdown
Member

@timmo001 timmo001 commented Apr 8, 2020

Proposed change

Screenshot_20200409_000632

Type of change

  • New feature (thank you!)

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

Comment on lines +161 to +163
srcset="
https://brands.home-assistant.io/${flow.handler}/icon@2x.png 2x
"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We removed these because we don't use them big enough to need it.

margin: 2px 0;
}
.discovered img {
width: 36px;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why not the same width as the integrations?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seems to take up quite a bit of space, but I can remove that so they match

Screenshot_20200415_152624

@timmo001 timmo001 requested a review from bramkragten April 15, 2020 14:28
@bramkragten
Copy link
Copy Markdown
Member

We are working on an overhaul of this page, so not sure how long this will stay.

@timmo001
Copy link
Copy Markdown
Member Author

I'll close this. #5580 supersedes this change.

👍

@timmo001 timmo001 closed this Apr 21, 2020
@lock lock Bot locked and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants