Skip to content

Svelte 5.19.5 breaks reactivity, Svelte 5.19.4 works fine #15149

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

Closed
patoi opened this issue Jan 30, 2025 · 4 comments · Fixed by #15158
Closed

Svelte 5.19.5 breaks reactivity, Svelte 5.19.4 works fine #15149

patoi opened this issue Jan 30, 2025 · 4 comments · Fixed by #15158
Assignees
Labels

Comments

@patoi
Copy link

patoi commented Jan 30, 2025

Describe the bug

In Svelte 5.19.4 reactivity works fine, but in 5.19.5 it's failing.

https://svelte.dev/playground/9a92552dd0204d93b4e76e4247103358?version=5.19.4

vs

https://svelte.dev/playground/9a92552dd0204d93b4e76e4247103358?version=5.19.5

Reproduction

https://svelte.dev/playground/9a92552dd0204d93b4e76e4247103358?version=5.19.5

Use case: Typing login name and password enables Login button.

In Svelte 5.19.5 it doesn't work: When I enter the login name and password, the Login button remains disabled. After I select the login name input field and type it again, the Login button becomes enabled.

Expected: When I enter the login name and password, the Login button becomes enabled. It works fine in Svelte 5.19.4

Logs

System Info

Svelte Playground

Severity

blocking all usage of svelte

@dummdidumm
Copy link
Member

FYI @trueadm this is likely due to a regression in #15137

@dummdidumm dummdidumm added the bug label Jan 30, 2025
@trueadm trueadm self-assigned this Jan 30, 2025
@trueadm
Copy link
Contributor

trueadm commented Jan 30, 2025

Looking into it now

@gyzerok
Copy link

gyzerok commented Jan 30, 2025

Interesting! Noticed that if you first touch password and then login, it works fine

@andriuwe4ka
Copy link

Just wrap in boolean and it works:

disabled={Boolean(!loginname ||! password)}

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

Successfully merging a pull request may close this issue.

5 participants