purescript-react / purescript-react-basic-dom

https://pursuit.purescript.org/packages/purescript-react-basic-dom
Apache License 2.0
11 stars 19 forks source link

Missing attributes with `Capture` suffix #51

Open i-am-the-slime opened 1 year ago

i-am-the-slime commented 1 year ago

I think all handler attributes (onClick, onChange, ...) have a corresponding attribute for registering handlers in the browser's capture phase (onClickCapture, onChangeCapture, ...).

We should add them.

codingedgar commented 1 year ago

Would it be as simple as adding onClickCapture here?

https://github.com/purescript-react/purescript-react-basic-dom/blob/a3406d6a27afcdea36a2dae7ab23245267a061a6/codegen/consts.js#L68

i-am-the-slime commented 8 months ago

Probably