kaliberjs / forms

Forms for React
MIT License
4 stars 0 forks source link

Added useBooleanFormField to the docs #5

Closed peeke closed 4 years ago

peeke commented 4 years ago

@EECOLOR Maybe we should rename useBooleanFormField to useCheckableFormField since it really only works if the connected input has a checked property?

EECOLOR commented 4 years ago

I see what you mean, although I think the numberFormField has the correct name and that one listens to onChange. So for now I think useBooleanFormField is fine, we might (in the future) want to be able to use useBooleanFormField with an onChange handler. But we'll cross that bridge when we get to it.