-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
chore: simplify updates #10922
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
chore: simplify updates #10922
Conversation
|
We can make our lives a lot simpler by replacing this...
...with this:
That might seem like a step backwards, but after minification it's a very minor difference of 3 bytes...
...and we're able to get rid of the
text_effect
function which minifies to 32 bytes (function a(b,c){d(()=>e(b,c()))}
), and which involves more function calls and more stuff that needs to be warmed up. The same is true for otherfoo_effect
functions likestyle_effect
andclass_toggle_effect
and so on (with more to come, hence draft status).Most significantly, it allows us to get rid of the distinction between 'singular' and 'grouped' updates in
template.js
:This results in a fairly substantial simplification of the compiler.
Before submitting the PR, please make sure you do the following
feat:
,fix:
,chore:
, ordocs:
.Tests and linting
pnpm test
and lint the project withpnpm lint