-
Notifications
You must be signed in to change notification settings - Fork 5
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
Material Design Icons support #51
Comments
The purpose here is to offer a way for the user to choose an icon from https://pictogrammers.com/library/mdi/ directly from the wizard! |
So I explorered a little what others did: 1/ Seems MQTT Hub introduced the first implementation to support MDI in a Homey application 2/ io.home-assistant.community took the code from MQTT Hub The implementation for both is based on Vue.js, but I use Petite Vue.js.
The icon file is downloaded and stored in the application file storage.
From user point of view it goes through a search tool inside the Homey interface
The interface doesn't show all the icons, but only the one selected
The code allows to input directly the url of a specific SVG file
If the same icon is used several times, the icon is duplicated, the file is named on the device unique identifier.
The file is deleted when the device is deleted.
I want to allow MDI for capabilities too,
|
First estimation of the work load: about 10h (dev) + 5h of support (bugs, wiki, ...). |
Ok so here is my idea after thikking about it a little while. In ESPhome firmaware, you can actually configure an icon at the entity level.
So the native capabilities will inherite from the icon of the entity. In Homey, you can set the icon of a device, but I'm unsure about the icon of a capability (only further test will confirm if it is possible). 1. The Wizard will propose for each device:
2. The Wizard will have a new "Manage your local icon library"
3. The download of the icons will be asynchronous
From an user experience point of view: I believe, if the user download an icon, he must have the choice to use it right now or later. From implementation point of view: |
Ronny already uses for his capabilities.
This was added in version 0.2.0 of his app.
This is a diff of version 0.1.9 and 0.2.0
RonnyWinkler/io.home-assistant.community@0.1.9...0.2.0
Originally posted by @RoadXY in #41 (comment)
The text was updated successfully, but these errors were encountered: