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.

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

Apply static vnode optimization to elements with event listeners #3518

Closed
Tracked by #3624
nolanlawson opened this issue May 18, 2023 · 1 comment · Fixed by #3519
Closed
Tracked by #3624

Apply static vnode optimization to elements with event listeners #3518

nolanlawson opened this issue May 18, 2023 · 1 comment · Fixed by #3519
Labels

Comments

@nolanlawson
Copy link
Collaborator

nolanlawson commented May 18, 2023

For an element like this:

<button type="button" onclick={onClick}>Click me</button>

...we should be able to apply the static vnode optimization. The only thing that makes it non-static is the onclick listener, but those are only mounted once anyway (never patched), so it should be possible to apply the optimization to this element.

@git2gus
Copy link

git2gus bot commented May 18, 2023

This issue has been linked to a new work item: W-13307351

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant