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

Deprecate forwardEvents and use:event action #84

Open
elcharitas opened this issue Mar 29, 2023 · 0 comments
Open

Deprecate forwardEvents and use:event action #84

elcharitas opened this issue Mar 29, 2023 · 0 comments

Comments

@elcharitas
Copy link
Owner

elcharitas commented Mar 29, 2023

Is your feature request related to a problem? Please describe.
Currently, we make use of an event drilling mechanism to pass events from a ChakraComponent to it's underlying HTML derivative.
This is made possible through the super amazing events prop that is native to the Box component only. This system works but it's a bit cumbersome to maintain and events may or may not get registered. I've also had to introduce lots of fixes to ensure events don't get registered passively.

Describe the solution you'd like
Once sveltejs/svelte#4599 or sveltejs/svelte#8356 gets approved and merged, we should migrate to using it instead and deprecate the use:event action. However, this would be a bit difficult because it means svelte versions in which on:* is not supported would not work with the latest releases of chakra. So we should prolly have both maintained and prolly disable event forwarding for specific svelte versions 😉

Describe alternatives you've considered
No other alternatives

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

No branches or pull requests

1 participant