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.

Sign up for GitHub

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

Jump to bottom

Svelte 5: Variables not referenced directly in "HTML" are not updated #9561

Closed
trygve-eide-igesund opened this issue Nov 20, 2023 · 0 comments · Fixed by #9581
Closed

Svelte 5: Variables not referenced directly in "HTML" are not updated #9561

trygve-eide-igesund opened this issue Nov 20, 2023 · 0 comments · Fixed by #9581
Assignees
@trueadm
Labels
bug
Milestone
5.0

Comments

@trygve-eide-igesund
Copy link

Describe the bug

Variables that only indirectly change the DOM through other variables are not updated correctly.

Reproduction

https://svelte-5-preview.vercel.app/#H4sIAAAAAAAAE4WS3U7DMAyFX8WKhLqJQYHL_gzxHBRVbeqWaJldJS7SVPXdSTcmQGrhKnJ88p0TJ6NqjUWvktdRUXVElaiXvlc7Jad-LvwHWsFQex6cnncyr53pZV9QIRYFHLbokDQ2ZWMcarEnyOEhvfaJpfxPo70v68qHNlP5-0DQRdEsDNJ2IC2GCbQ1-rDZwjjvFrKEv83hMb201xL8kPwdYA1wA0-QBwg8Q1RX-tA5Hqi502zZJWEwTQTJV_rZg8mzxXvL3WaBtluz2aYQx0zBbymD8WBIO6y8oW42mgrK4u83oqweRMLMmJLz2PLxvEzg5WQxVOtXn_ZXNkLN8g5ZfIEF8LgQJliHn3LkxrQGG5WIG3B6mz4BBCQakGQCAAA=

I expected the button to switch between red and not red every other click.
And that referenced_directly and not_referenced_directly would increase, but only the first one is.

Logs

No response

System Info

svelte-5-preview.vercel.app

Severity

blocking an upgrade
@trygve-eide-igesund trygve-eide-igesund changed the title Svelte 5: Variables not referenced directly in "HTML" is not updated Svelte 5: Variables not referenced directly in "HTML" are not updated Nov 20, 2023
@trueadm trueadm self-assigned this Nov 20, 2023
@benmccann benmccann added the bug label Nov 20, 2023
@benmccann benmccann added this to the 5.0 milestone Nov 20, 2023
@trueadm trueadm mentioned this issue Nov 21, 2023
Merged
dummdidumm pushed a commit that referenced this issue Nov 21, 2023
@trueadm
fix: adjust event delegation heuristics (#9581)
da15806 
Fixes #9561
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trueadm trueadm

Labels
bug
Projects
None yet
Milestone
5.0
Development

Successfully merging a pull request may close this issue.

Fix delegation bug sveltejs/svelte
3 participants
@benmccann @trueadm @trygve-eide-igesund