docs: Recommend deriving over syncing state #15994
Conversation
The main change treats spent as the source of truth and derives left from it—simplifying the code, reducing the risk of inconsistencies, and improving the clarity of their relationship. I also changed let total = 100 to const total = 100 to more clearly convey that the value is intended to remain constant.
|
|
This is great, thanks! We can go a step further though — since Since GitHub's suggestions UI is cumbersome for things that span multiple edits, I'll push up some changes |
When not to use $effect section
Closes: #15955
Following the discussion on this issue #15955 , @elliott-with-the-longest-name-on-github encouraged me to submit a PR to update the "money spent / money left" example in the “When not to use
$effect” section of the docs.The main change treats
spentas the source of truth and derivesleftfrom it—simplifying the code, reducing the risk of inconsistencies, and improving the clarity of their relationship.I also changed
let total = 100toconst total = 100to more clearly conv 8000 ey that the value is intended to remain constant.Let me know if I missed anything, I'll fix it right away. Thanks!
Before submitting the PR, please make sure you do the following
feat:,fix:,chore:, ordocs:.packages/svelte/src, add a changeset (npx changeset).Tests and linting
pnpm testand lint the project withpnpm lint