bug: All @Events are cancelable by default when component may not necessarily support event cancelation
Prerequisites
- [X] I have read the Contributing Guidelines.
- [X] I agree to follow the Code of Conduct.
- [X] I have searched for existing issues that already report this problem, without success.
Stencil Version
2.17.1
Current Behavior
Any custom event created with @Event is set as cancelable: true by default. Developers should explicitly make an event cancelable once a component supports cancelation.
One side-effect from this is that anything derived from the docs-json output target can be misleading. For example, we use the docs-json output file to create custom doc pages and there was some confusion about event cancelation as they were all documented as such, but only a handful were truly cancelable (see https://github.com/Esri/calcite-components/issues/3967 for more info).
Expected Behavior
Events are not cancelable by default.
Steps to Reproduce
See steps in readme from https://github.com/jcfranco/stencil-all-events-cancelable-by-default
Code Reproduction URL
https://github.com/jcfranco/stencil-all-events-cancelable-by-default
Additional Information
No response
Hey @jcfranco 👋
Sorry for the delay on this. I was able to confirm we default events as cancelable and that is getting reported in the generated documentation JSON files. I'll get this labeled so it's pulled into our backlog and we can discuss as a team. Thanks for reporting the issue!