qwik icon indicating copy to clipboard operation
qwik copied to clipboard

[📖] Ecosystem - replace @qwikest/icons with icônes.js

Open thejackshelton opened this issue 2 years ago • 5 comments

Suggestion

On the Qwik Ecosystem page one of the top libraries is @qwikest/icons. While everyone loved the DX of the package, it hasn't been working since the 1.0 release to my knowledge.

I've had several conversations with Qwik community members that have been flustered with the icon library. In each conversation I've been recommending the use of https://icones.js.org/ instead, due to the Qwik integration and availability of icons.

From what I understand the creator of the package is working on several other projects outside of it, and it will be a while before he gets back around to it.

If this package could be removed from the ecosystem page with icônes in its place that would be much appreciated. Then when the author or any contributors get the package working again we can make a PR for it as an integration.

Discord Channel References: https://discord.com/channels/842438759945601056/1105851336169357374 https://discord.com/channels/842438759945601056/1106222651355176960 https://github.com/qwikest/icons/issues/10 https://github.com/qwikest/icons/issues/11

thejackshelton avatar May 13 '23 05:05 thejackshelton

FYI we just fixed the entrypoint and package size problems in version 0.0.8. I'm sorry for the inconveniences caused 😢 Let's see if everything is going fine now. manus suggestion helped a lot 👀

NiklasPor avatar May 13 '23 06:05 NiklasPor

hey jolks!

is it working now? i am fine to rearrange in the docs, not strong opinions here. The docs for icons are important when people search, or Qwik AI to provide some reasonable answer :)

manucorporat avatar May 13 '23 08:05 manucorporat

Yes it is working now. I just test it with qwik (latest 1.1.2 & 0.107.0) and @qwikest/icons (v0.0.8) + tailwind too.

There's still the warnings but idk if they are actually from qwik. Anyhow, now icons are ready to go. Lots of thanks to @NiklasPor for all the hard work!

image image

da8ah avatar May 13 '23 15:05 da8ah

Other libraries are also facing the warnings, so it's at least not specific to this one. There's also an issue inside the qwik repo tracking it

NiklasPor avatar May 13 '23 15:05 NiklasPor

Ah I didn't realize it was currently being worked on! @NiklasPor thank you for putting the time in to fix this. I know it was unexpected but we really appreciate your package. I'll make sure to mention that it's working in 1.1.2.

thejackshelton avatar May 13 '23 15:05 thejackshelton

@thejackshelton can we close this as resolved? 🙏

zanettin avatar May 18 '23 20:05 zanettin

yup!

thejackshelton avatar May 18 '23 20:05 thejackshelton