iconify / icon-sets

150+ open source icon sets. Icons are validated, cleaned up, optimised, ready to render as SVG. Updated automatically 3 times a week.
https://icon-sets.iconify.design/
591 stars 56 forks source link

HeroIcons update issue #54

Closed chasegiunta closed 1 year ago

chasegiunta commented 1 year ago

HeroIcons recently released v2.0.0 of their icon package.

Unfortunately, this update was major (changed the stroke width, the style of the icons, introduced a newer size, etc). In v1.0, the icon identifiers were prefixed as heroicons-outline:icon & heroicons-solid:icon... The prior remains untouched and still shows the v1.0 icons (for now, but looks like you removed heroicons-outline https://github.com/iconify/icon-sets/commit/d5ebd94075f1fd65fdd6fadd358604d688ffa8bb, does this mean these icons will disappear??)

The latter was updated overnight to 2.0 and the change has surely by now propagated to hundreds of sites without warning.

Surely there has to be an upgrade path or something, not just removing and replacing entire icon sets?

cyberalien commented 1 year ago

This was indeed a major change. In old version solid was 20x20, outline was 24x24. In new version there are 3 types of icons:

That created a bit of mess. Also new outline version is different than old one.

So I've updated heroicons set instead. It now contains both 24x24 variations. heroicons-solid contains updated 20x20 icons, heroicons-outline contains version 1 of 24x24 outline icons.

edit: fixed comment

chasegiunta commented 1 year ago

heroicons-solid contains updated 20x20 icons, which seem to be same as old ones (probably kept for legacy support)

Unfortunately, they are not, which is how I woke up to notice the change in our icon styles on our production sites.

So, your recommended solution is quickly go through every codebase that uses heroicons-solid and update it to heroicons and append -solid to the end of the name, and deploy as fast as we can?

I hope you understand why I'm asking that updates to iconsets now and in the future should not be done this way. Is there a way to keep version ^1.0 icons so we can adequately update designs or have the time to work out an upgrade path?

(I understand Iconify is a free service and I'm incredibly appreciative of the work you do, just hoping to avoid this scenario in the future)

cyberalien commented 1 year ago

Doh.

I think this needs reversing to v1 for heroicons-solid. Not sure yet what to do with new 20x20 solid version...

cyberalien commented 1 year ago

Reversed heroicons-solid. It also includes new icons to avoid breaking things for users who started using icon set in last few days. For icons that exist in both v1 and v2, it uses v1.

Now need to figure out where to put v1 of 20x20 solid icons.

NPM packages are updated, but API and big icon sets package are not updated yet. They'll be updated after I fix the rest of it.

chasegiunta commented 1 year ago

I wonder if the move going forward with updates like this is the same as what you've done with Font Awesome, eg:

HeroIcons 1.0 HeroIcons 2.0
heroicons-outline:check hi2:check
heroicons-solid:check hi2:check-solid
hi2:check-mini
cyberalien commented 1 year ago

Good idea and that's sort of how it is right now. heroicons-outline and heroicons-solid are both v1 and will remain as is, in "Archive / Unmaintained" category similar to FontAwesome 4/5.

cyberalien commented 1 year ago

Solved it. heroicons now contains all icons. 20x20 solid icons use suffix -20-solid. Also it supports possibility of 20x20 outline icons if they'll be added in future.

API should be updated in few minutes, website in about an hour.

Thanks a lot for reporting it!

cyberalien commented 1 year ago

Everything has been updated.

v2: https://icon-sets.iconify.design/heroicons/ v1 solid: https://icon-sets.iconify.design/heroicons-solid/ v1 outline: https://icon-sets.iconify.design/heroicons-outline/

cyberalien commented 1 year ago

Another minor bug caused by fix: v1 icon sets are currently marked as hidden. It doesn't affect their functionality, so all icons work as expected, only causes them to not show up in search results and when browsing icon sets.

It will be fixed on next update.