-
Notifications
You must be signed in to change notification settings - Fork 13
Why don’t you declare the icons as ES Modules, so they’re fully tree-shakable? #51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our 8000 terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I don’t like multiple imports for icons but i can add this feature later. Example usage import UserIcon from "@monicon/lucide/user"
import AccountIcon from "@monicon/lucide/account" |
thanks, it would be great |
@oktaysenkan This would be far more preferable than adding individual icon names to the build config or unnecessarily bloating the bundle with an entire collection. I'd love to recommend we adopt Monicon within my workplace, as the standard icon solution for all our apps, but unfortunately I can't do that if the DX every time is "use an icon.. check the config" |
I am thinking a solution for this. Metro does not provide module load function like Vite. |
I just created the issue on metro repo |
I will build a CLI tool to automate the generation of icon files in different formats ( Core Features:
This will streamline icon management and improve workflow efficiency. What do you think? |
You wouldn’t have to deal with the hassle of manually adding the icons you’re going to use in the vite.config.js.
Even if your library has millions of icons, all of them would be available during development, but only the ones actually used would be included in the final build shipped to the client.
The text was updated successfully, but these errors were encountered: