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

Svelte 5 is breaking sites (including the Svelte 5 preview site) #13340

Closed
Ocean-OS opened this issue Sep 19, 2024 · 5 comments
Closed

Svelte 5 is breaking sites (including the Svelte 5 preview site) #13340

Ocean-OS opened this issue Sep 19, 2024 · 5 comments
Labels
Milestone

Comments

@Ocean-OS
Copy link

Ocean-OS commented Sep 19, 2024

Describe the bug

When going to the svelte 5 preview site, it flickers and then shows nothing other than a gray screen. When checking devtools, it has this error:
image
It also appears to affect other Svelte 5 sites, as shown by @itsneeku.

Reproduction

Go to the Svelte 5 preview site.

Logs

class.CTOAjPUp.js:1 
 Uncaught (in promise) TypeError: Cannot set property width of #<SVGSVGElement> which has only a getter
    at z (class.CTOAjPUp.js:1:3883)
    at focus-outside.BZqWhF3q.js:1:1248
    at Mt (runtime.CZX3LrFh.js:1:7787)
    at j (runtime.CZX3LrFh.js:1:8567)
    at b (runtime.CZX3LrFh.js:1:3689)
    at z (runtime.CZX3LrFh.js:1:4567)
    at Ir (runtime.CZX3LrFh.js:1:4599)
    at N (focus-outside.BZqWhF3q.js:1:1241)
    at external-links (0.CRbYljr2.js:1:31135)
    at q (theme.Ci2GTQxT.js:1:150)

System Info

Tested on Chromium (latest) and Microsoft Edge (latest).

Severity

blocking all usage of svelte

@itsneeku
Copy link

Its affecting static builds deployed on gh pages as well: https://neeku.dev/concordia-ics-builder/

@Ocean-OS Ocean-OS changed the title Svelte 5 Preview site is broken Svelte 5 is breaking sites (including the Svelte 5 preview site) Sep 19, 2024
@o-az
Copy link

o-az commented Sep 19, 2024

I’m using this deployment from 30 days ago until current svelte 5 docs are fixed

https://svelte-5-preview-iu6hmd250-svelte.vercel.app/docs/introduction

@Conduitry
Copy link
Member

More broadly: #13327 doesn't appear to account for cases where the DOM property exists but is read-only.

@Conduitry Conduitry added the bug label Sep 19, 2024
@Conduitry Conduitry added this to the 5.0 milestone Sep 19, 2024
@Ocean-OS
Copy link
Author

More broadly: #13327 doesn't appear to account for cases where the DOM property exists but is read-only.

After looking through the built code I'm pretty sure the function causing the error is here.

@Conduitry
Copy link
Member

This has been fixed in #13341.

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

No branches or pull requests

4 participants