Skip to content
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 terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate dynamic imports for icons-react #1081

Merged
merged 2 commits into from
Sep 27, 2024
Merged

Conversation

timheerwagen
Copy link
Contributor

fixes #998

This PR generates a dynamic import for every Icon, so it can be used for lazy loading.

Also i had to use slash when building, because Windows could not read the joined path.

Copy link

vercel bot commented Apr 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
tabler-icons ✅ Ready (Inspect) Visit Preview Sep 26, 2024 11:44pm

Copy link

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 github-actions bot added the stale Lack of activity label May 23, 2024
@timheerwagen
Copy link
Contributor Author

Will this PR be merged, or should I make some changes?

@github-actions github-actions bot removed the stale Lack of activity label May 24, 2024
@devcshort
Copy link

I NEED this for my current project. Please merge it whoever needs to :'(

Copy link

github-actions bot commented Jul 1, 2024

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 github-actions bot added the stale Lack of activity label Jul 1, 2024
@timheerwagen
Copy link
Contributor Author

Still interested in this feature.

@github-actions github-actions bot removed the stale Lack of activity label Jul 1, 2024
Copy link

github-actions bot commented Aug 1, 2024

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 github-actions bot added the stale Lack of activity label Aug 1, 2024
@codecalm codecalm removed the stale Lack of activity label Aug 8, 2024
Copy link

github-actions bot commented Sep 8, 2024

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 github-actions bot added the stale Lack of activity label Sep 8, 2024
@aminelemaizi
Copy link

aminelemaizi commented Sep 8, 2024

I would love to have this feature, any chance to integrate this in the next release?

@github-actions github-actions bot removed the stale Lack of activity label Sep 9, 2024
@gDlugokecki
Copy link

Is there a chance somebody could merge this? This is valuable and highly desired!

@codecalm codecalm merged commit 7d1f101 into tabler:main Sep 27, 2024
2 of 3 checks passed
@codecalm
Copy link
Member

thank you for your help! ❤️

@KevinVandy
Copy link

Since this is now crashing people's machines in development mode, can this PR be reversed? It's not fun having people yell at me every day that this package breaks there dev server just because I recommended it as a peerDep.

@nicosammito
Copy link

Since this is now crashing people's machines in development mode, can this PR be reversed? It's not fun having people yell at me every day that this package breaks there dev server just because I recommended it as a peerDep.

Would be a good idea to reverse this, because we have problems too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get Icon dynamically
7 participants