-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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 : with reactive set has
invokes in effects where it shouldn't
#11515
Comments
If it doesn't matter can we consider #11504? |
#11287 also fixes the problem described here. |
9 tasks
I want to expand on this issue.
it seems like tracking version here is wrong. |
a better description is provided in #11727 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
With
ReactiveSet
everything is fine grained, for instance doing this:If you press the "delete 2" button, it only logs "has 2: false". But if the value is not there to begin with?
now if you press the "delete 2" button, both "has 2: false" AND "has 3: false" will get logged, it also happens when you add something new.
It might be because of these lines? When a change occures we increase the version, which makes the call to "has 3: false" to be invoked.
Reproduction
1- go this REPL
2- press on "delete 2" button
3- see the logs which also logs "has 3: false"
4- press on "add 2" button
5- see the logs which also logs "has 3: false"
Logs
No response
System Info
Severity
annoyance
The text was updated successfully, but these errors were encountered: