hass-fontawesome icon indicating copy to clipboard operation
hass-fontawesome copied to clipboard

Remove trailing slash in custom icons path

Open cascer1 opened this issue 3 years ago • 1 comments

The trailing slash prevents loading of custom icons

cascer1 avatar May 26 '21 08:05 cascer1

while I am also looking for a way to change the default path custom_icons, to a path in my /local folder (www), I can confirm the original CUSTOM_ICONS_PATH = 'custom_icons/' does work as advertised with custom .svg icons.

Mariusthvdb avatar Sep 29 '21 10:09 Mariusthvdb

@thomasloven Are you still monitoring this repo? Would you mind showing it some love and accepting some of the PR's which address these sort of issues? I'm also unable to use custom .svg's using the instructions you provided. Thank you.

GitHubGoody avatar Feb 27 '23 20:02 GitHubGoody

@cascer1 and @Mariusthvdb

This is the only line I've changed in my __init__.py: CUSTOM_ICONS_PATH = 'custom_icons'

But, I'm still not able to use custom icons placed in that folder. Did you have to do anything else to get this to work?

Also, it doesn't seem that @thomasloven is maintaining this Repo anymore. Any interest in forking it and accepting the multiple outstanding PRs? Thanks

GitHubGoody avatar Mar 20 '23 15:03 GitHubGoody

I must admit that I'm no longer using this integration, so I probably won't be the best person to maintain a fork.

cascer1 avatar Mar 21 '23 07:03 cascer1

@cascer1 What are you using instead for custom icons?

GitHubGoody avatar Mar 21 '23 14:03 GitHubGoody

depending in your requirements, you might able to use my custom_icons, see my repo. there's a CC allowing you to drop in any svg, and a custom-card that has icons builtin. persoanlly I like the CC best. It's a plain fork of Thomas's repo, without the options for fontawesome

Mariusthvdb avatar Mar 21 '23 14:03 Mariusthvdb

@cascer1 What are you using instead for custom icons?

I use https://github.com/arallsopp/hass-hue-icons for some icons for my lights, and otherwise don't need custom icons currently.

cascer1 avatar Mar 21 '23 15:03 cascer1

I cannot reproduce this issue...

thomasloven avatar Oct 14 '23 20:10 thomasloven

I was about to add "...but it also doesn't hurt..." and just merge this. But then I actually tested it, and MY WHOLE COMPUTER IMMEDIATELY CRASHED. I kid you not.

Merged!

thomasloven avatar Oct 14 '23 21:10 thomasloven