Don't store `JsValue` in `ListenerProduct`

maciejhirsz / kobold

Easy declarative web interfaces.

https://docs.rs/kobold/

Mozilla Public License 2.0

385 stars 7 forks source link

Closed maciejhirsz closed 1 year ago

maciejhirsz commented 1 year ago

No need to store the JsValue for a closure, it's fine to just pass it into the compiled JS code by value.
No need to update bound event listeners if the closure they are created from is zero-sized.

This results in a minor reduction in the Wasm file (TodoMVC goes from 35173 bytes to 34986 bytes, uncompressed).