Vjacheslav Trushkin

> @cyberalien does this icon https://www.svgviewer.dev/s/ottD7T7J follow iconify json convention?, if I just change the svg to use `viewBox="0 0 24 25"` removing width and height attributes it just works:...

Yes, test case shows bug in utils, so it should be fixed there. I'll fix that

Actually... won't fixing that bug break Unplugin Icons? Same loader is shared with Unplugin Icons, I suspect expected behavior there is to keep dimensions from custom icon, not replace them...

I'm not sure changing code in utils is correct way to fix it. In loader this is intended behavior. Function `mergeIconProps` checks if width and height are set on SVG,...

> @cyberalien How about making it optional? I tried to fix it, also considered option to make it optional, but loader code is very confusing, I'm afraid that fixing things...

This is a tricky issue. Current implementation is by design. `IconifyIcon` type is not designed to handle additional attributes, attributes are intended to be removed from SVG element. **Why is...

Need to think of a temporary solution for this issue. Proper solution would require changing type, but simply adding extra attributes to type is not a good idea because I'd...

Unfortunately I don't think it would. Can't figure out how to make it work. Icons are converted to icon set in `IconifyJSON` format, which does not support extra attributes. Even...

That function lists icons that are loaded. If it is ran after rendering all pages, which should load icon data, it might work for your use case. Docs: https://iconify.design/docs/icon-components/vue/list-icons.html

I tried, but can't figure out how to animate it :( Though it was a while ago, since then I did figure out few new methods to animate stuff, so...