Replies: 25 comments 17 replies
-
Beta Was this translation helpful? Give feedback.
-
|
when you post in threads always tag me because I don't get notifications |
Beta Was this translation helpful? Give feedback.
-
|
@elax46 Also I noticed from the iconfy site some of icons were too small and off to the left (I tried them in my preview manager and they showed up like this, I have donwloaded the ones that are like this into a folder and everytime I do a set ill do 10 to 15 corrections on them and keep file names the same so you know which ones to change. |
Beta Was this translation helpful? Give feedback.
-
ok I kindly ask you to separate the icons into two different zips, one for modifying and the other for adding new ones |
Beta Was this translation helpful? Give feedback.
-
|
260284.zip |
Beta Was this translation helpful? Give feedback.
-
|
Valid for the next version they have already been added |
Beta Was this translation helpful? Give feedback.
-
|
Also you may want to look at the coding as it seems other icon and repos have adjusted theres lately ours currently looks like this
were theres would look like this and might be something needed to adapt to fix some issues
|
Beta Was this translation helpful? Give feedback.
-
|
Thanks for your work as always, but I didn't understand what was changed compared to the other projects |
Beta Was this translation helpful? Give feedback.
-
|
by the looks of it, its just the spacing of the in the java, but also if you open our js and compare to others in visual its spaced differently too on the code side, it might be nothing but I have compared to a few files now and they all our simular. ours is different |
Beta Was this translation helpful? Give feedback.
-
|
it's just a question of indentation it's nothing functional but just to make the code "nicer to look at" |
Beta Was this translation helpful? Give feedback.
-
|
@rchiileea Personal request :) in the next version could you create two small icons of the P1S and A1 Mini bambulab? I need them as icons for smart sockets |
Beta Was this translation helpful? Give feedback.
-
|
@elax46 you mean something like this? |
Beta Was this translation helpful? Give feedback.
-
|
That's great! |
Beta Was this translation helpful? Give feedback.
-
|
|
Beta Was this translation helpful? Give feedback.
-
|
it's great! Thank you |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
|
|
Beta Was this translation helpful? Give feedback.
-
|
all the fixed ones are I forgot to remove the surround... doh, so expect 20 in new update of fixed icons :( doh |
Beta Was this translation helpful? Give feedback.
-
|
@elax46 see above post lol, here they all are fixed and then new ones added, if at all possible can we get these icons tonight or in the morning so I can share tomorrow on release party (contain quite a few stairs icons for the new floors introduced in 2024.4) |
Beta Was this translation helpful? Give feedback.
-
|
toofix.zip |
Beta Was this translation helpful? Give feedback.
-
|
|
Beta Was this translation helpful? Give feedback.
-
|
Having looked up the iconify project I repaired the ones that showed off to the left in that (and in many other apps when you use iconfy as that has a way of showing faults)... if you look through the list https://icon-sets.iconify.design/cbi/ you will see only the ones showing small and to the left upper corner and the ones remaining in the folder too fix i sent you. although I have noticed some of the ones I sent previously have not been updated into our repo either (sockets, signee etc) So I will have to go through that list again once you have updated all the too fix items, there are more though I notice (but once you have added all the too fix I can find them) as hovering now shows sizes and there are quite a few showing over 24x24 so more to fix, I can not try to fix until we have all the latest ones added, then I can then go through the oversized ones and fix them. Edit, I have noticed as well just going through the ones I think need redoing (over sized ones due to view box being between 32x32 and some even 5000x5000 view box lol... and some are mine, but others are user pull requests. |
Beta Was this translation helpful? Give feedback.
-
|
p.s get the owner of the iconify repo to add by Emanuele and rchiileea :P |
Beta Was this translation helpful? Give feedback.
-
|
adore-alt.zip |
Beta Was this translation helpful? Give feedback.
-
|
@elax46 these icons in the list need changing, but luckily you only need to change view box in the file viewbox changes in json file all changed to 0 0 24 24 sync-box only view box needs changing as they sit to the left in apps using iconify and your web app, these have to be changed. then the bad news Then their should only be 80 odd that will need resizing due to the fact they range from 128 128 viewbox up too (which shcked me) 5000 5000 view box (and explains why some of the icons are not visible in your web based app at all, I expect the 32x23 ones are the ones that only show part of the icon).... but these could be a explination of why people have caching issues too. So first correct the icons listed above to view box 0 0 24 24 do not correct any others, then change your web app view box to 0 0 32 32 Then I will go through the list again and download all the files above that to correct, once this is done If we get any caching issues of icons from users moving forward I will have to resize the ones that are 0 0 32 32... But I am hoping the HA app will accept that size and i dont have to as 50 icons to correct is much better than 214. |
Beta Was this translation helpful? Give feedback.
-
|
thanks for your detailed analysis, I will fix the viewboxes of the icons you indicated and check the issue on the icon site to understand if I can modify the code in relation to the icon viewbox without setting a static viewbox |
Beta Was this translation helpful? Give feedback.
-
|
update: I corrected all the icons you indicated, I also updated the code to find the icons with the viewbox and at 32x32 it seems to work for many icons but several still look bad so they need to be modified |
Beta Was this translation helpful? Give feedback.
-
|
icons with problems:
|
Beta Was this translation helpful? Give feedback.
-
|
lol I am going to busy, thanks for saving me the time of going through them, I knew there would be a few. |
Beta Was this translation helpful? Give feedback.
-
|
take all the time you need |
Beta Was this translation helpful? Give feedback.
-
|
back from break, so i went through again my self and found 130 that need adjusting/fixing in total lol |
Beta Was this translation helpful? Give feedback.
-
|
so been busy, I had to redo a lot of these completely as they had way to many points some had 1000 points, all the arlo stuff has been redone, fixed points on some of the other stuff and redone all the battery ones, redone the apple ones too only 75 to go |
Beta Was this translation helpful? Give feedback.
-
|
2 hours of work! Anyway, thank you always for your support |
Beta Was this translation helpful? Give feedback.
-
|
latest requests and redone |
Beta Was this translation helpful? Give feedback.
-
|
@elax46 omg I have been busy in spare time, so many icons I actually redone most of them too so they had much few anchors, some that were over 1mb are now 2kb... some like the sonos obviously could not be changed due to the grill effect they have. I then had a blast from the past redoing the moes ones as those are icons I requested after donating a long time ago now, way before i taught myself to do icons lol.. new icons are will be uploaded tomorrow hopefully this will help with caching and sizing issues for your web icon browser |
Beta Was this translation helpful? Give feedback.
-
remarkable work for both |
Beta Was this translation helpful? Give feedback.
-
|
From the icon creator it seems that there are still some icons that are not visible @rchiileea
|
Beta Was this translation helpful? Give feedback.
-
29-01-24.zip
Beta Was this translation helpful? Give feedback.
All reactions