Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Consider onBlur, onInput and other form field specific events for DateInput, Radios and Checkboxes #29

Open
andymantell opened this issue Jan 17, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@andymantell
Copy link
Collaborator

Consider onBlur, onInput and other form field specific events for DateInput, Radios and Checkboxes. Since these components are effectively "compound" fields, we don't by default spread the top level props into the individual inputs (Since they are built from the items prop). This means that putting top level onBlur handlers in doesn't actually work. onChange works because I have specifically catered for this, but we should consider other events.

People can inject them via the individual items, but I think it would be cleaner not to require people to go to this extra effort.

Note: Make sure that any handlers specified on the individual items aren't wiped out by the top level one (i.e. insert the top level one and spread the individual item attributes afterwards)

@andymantell andymantell added the enhancement New feature or request label Feb 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant