lucide icon indicating copy to clipboard operation
lucide copied to clipboard

Add Magento 2 Hyva package to docs

Open GrimLink opened this issue 2 years ago • 8 comments

I love what you all doing with this Icon pack, I made a Magento 2 module for Hyva Themes that uses Lucide -> https://github.com/Siteation/magento2-hyva-icons-lucide

This PR will add the icon to homepage header, and adds a page to to the docs.

GrimLink avatar Jun 16 '22 16:06 GrimLink

I am not sure how to add a section to the package page, not clear how this is handled, I suspect this done trough each package.json in the packages folder, is there an easy way for 3de part modules to be added to this list?

But anyway that's is something for later to add.

GrimLink avatar Jun 16 '22 16:06 GrimLink

@ericfennis any chance you or someone else can take a look at this PR?

GrimLink avatar Jun 28 '22 16:06 GrimLink

Sorry for the late reply. I appreciate the work, nice package. I'm not sure if we need to place "not official lucide" packages on the homepage to avoid confusion among users. I now laravel there as wel but I think we should move that one as wel. I think it is better if we make a "third party packages" section on the packages page.

ericfennis avatar Jun 29 '22 20:06 ericfennis

@ericfennis any chance this can be added now and moved later? I would love to tweet and do some more things about this addition to Hyva, as alternative icon pack.

Also I think the icons on the homepage are not confusing, the docs and the packages are the best place to split the 3de party option from the core options.

GrimLink avatar Jul 01 '22 08:07 GrimLink

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Aug 17 '22 03:08 github-actions[bot]

@ericfennis is there anything I can help with?

GrimLink avatar Aug 17 '22 11:08 GrimLink

@GrimLink Sorry for the late reply and lack of help with this implementation. It's a bit on a low prio right now.

If you like to add it you can, we want to create a third-party section on the packages page on the site. We can create a json file for it to specify the package name and etc. We already have an json file for the first party packages, so we can base it on that one.

ericfennis avatar Aug 20 '22 21:08 ericfennis

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Sep 20 '22 04:09 github-actions[bot]

@GrimLink I've added https://github.com/lucide-icons/lucide/blob/main/site/src/data/packageData.thirdParty.json, a data list for third-party packages. It will show up on https://lucide.dev/packages.

ericfennis avatar Oct 24 '22 16:10 ericfennis

@ericfennis super! I have not got a chance to to take a look to improve the 3de party implementation. So this great.

I will create a new PR with with only the change to the packageData.thirdParty.json 😉

GrimLink avatar Oct 26 '22 10:10 GrimLink