[@mantine/hooks] use-timeout: Memoize clear
and start
functions
#3801
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Solves #3789
clear()
andstart()
functions with theuseCallback()
hook so that they won't change across re-renders (unless they have to, e.g.start()
depends ondelay
).autoInvoke
effect whendelay
changes.autoInvoke
effect dependencies to includeclear()
andstart()
as well asoptions.autoInvoke
.As far as I could check, the above changes shouldn't affect anyone using this hook. However, there's a separate commit (see point 3) that changes
autoInvoke
effect dependencies and I'm not 100% sure of relying onoptions.autoInvoke
. People shouldn't really change that setting after the first render, so I'd think it's safe, but I'm open to suggestions.