Make input fields easier to focus even without FormControl wrapping #97
Conversation
🦋 Changeset detectedLatest commit: aaf0b69 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| {...props} | ||
| ref={ref} | ||
| placeholder=" " // This is needed to make the label work as expected | ||
| /> | ||
| <FormLabel>{label}</FormLabel> | ||
| <FormLabel htmlFor={id} pointerEvents="none"> |
There was a problem hiding this comment.
Man kan argumentere for at pointerEvents="none" på en label er feil, men i dette tilfelle vil resultatet bli det samme om man klikker på labelen eller ikke. Hele labelen vil alltid være på innsiden av input-feltet, så da ser jeg ikke noe problem med å gjøre det slik.
Alternativt så kunne man brukt de interne hookene useFormControlProps og useFormControl for å hente ut riktige IDer for å binde dem sammen, men det har sin egen risikoprofil (de er ikke dekket av semver f.eks). Så summa summarum, et reflektert trade-off.
This fixes a bug where - if you didn't wrap your
<Input />component in a<FormControl />component, and didn't provide your own ID - you couldn't click the overlaying label to focus the input.