-
Notifications
You must be signed in to change notification settings - Fork 29
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
svelte 5 support #115
Comments
@ankurrsinghal I haven't had any time to look into this, have you had any chance to try out svelte 5, release is coming really soon |
it looks like reverting #80 should help resolve a whole bunch of issues since in Svelte 5 use of internal functions is now an error [sveltejs/svelte/pull/11632] |
Svelte 5 is here! This package breaks my app since it still imports from svelte/internal. If it can be updated that would be great. |
Pull request #124 just got merged, that should fix the main issue, can anyone confirm by building from source? |
@IsraPerez98 I can't get it to build from source, it would be really great if @ankurrsinghal could chime in here. I wouldn't mind opening a PR to work on this, I just don't want to duplicate effort. |
looks like we need an npm release for that merge |
Waiting for some kind of fix was taking too long, so I did it myself with #125. |
It looks like this has been merged into master, it just needs an NPM update (@ankurrsinghal, when you can, this would be most welcome. Thanks!) |
If you create a new svelte 5 starter app (out of the box supported with create app..) and add svelte legos, it completely breaks the site. I'm really only using the media queries and clipboard.
The text was updated successfully, but these errors were encountered: