Typed event data + async event handlers

[lukechu10]

Closes #241 and #506

Async event handlers somehow do not type check yet.

This is a breaking change because closures now accept different types as arguments depending on the event used. This is also a breaking change because invalid event names are no longer allowed, and the event structs must be used explicitly when using the builder API.

[codecov]

Codecov Report

Base: 63.28% // Head: 63.09% // Decreases project coverage by -0.19% ⚠️

Coverage data is based on head (6e52e49) compared to base (9c5c12e).
Patch coverage: 29.23% of modified lines in pull request are covered.

❗ Current head 6e52e49 differs from pull request most recent head b59589c. Consider uploading reports for the commit b59589c to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #553      +/-   ##
==========================================
- Coverage   63.28%   63.09%   -0.20%     
==========================================
  Files          54       55       +1     
  Lines        8952     8995      +43     
==========================================
+ Hits         5665     5675      +10     
- Misses       3287     3320      +33     

Impacted Files	Coverage Δ
packages/sycamore-core/src/event.rs	0.00% <0.00%> (ø)
packages/sycamore-core/src/generic_node.rs	75.00% <ø> (ø)
packages/sycamore-core/src/lib.rs	100.00% <ø> (ø)
packages/sycamore-macro/src/view/ir.rs	100.00% <ø> (ø)
packages/sycamore-router/src/router.rs	27.88% <0.00%> (ø)
packages/sycamore-web/src/dom_node.rs	0.00% <0.00%> (ø)
packages/sycamore-web/src/hydrate_node.rs	0.00% <0.00%> (ø)
packages/sycamore-web/src/lib.rs	21.42% <ø> (ø)
packages/sycamore/src/builder.rs	24.51% <0.00%> (ø)
packages/sycamore/src/web/html.rs	69.36% <0.00%> (-0.30%)	⬇️
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[piaoger]

@lukechu10
I am not sure if it's right place or time for my question :) ev seems so short for naming, maybe evt is better.

[lukechu10]

@lukechu10 I am not sure if it's right place or time for my question :) ev seems so short for naming, maybe evt is better.

We can still change it but I am personally in favor of ev. It is quite common to see "ev" for an abbreviation of "event" in event handlers.

[piaoger]

We can still change it but I am personally in favor of ev. It is quite common to see "ev" for an abbreviation of "event" in event handlers.

@lukechu10
Thanks for your effort on Sycamore 👍
I have no problem now and then I can update my indoor app to align with latest changes in main branch :)