miljodir / md-components

Design-komponenter for Miljødirektoratet (CSS og React)

https://miljodir.github.io/md-components

MIT License

3 stars 2 forks source link

Ref on MdTextArea #110

Closed kajsaeggum closed 7 months ago

kajsaeggum commented 7 months ago

Describe your changes

Added a forwardRef to MdTextArea to be able to focus on the inner text area through use of ref's. Specific use case is when there are form validation errors and the text area should be focused. (Same functionality is in place for MdInput and MdSelect)

Checklist before requesting a review

[x] I have performed a self-review and test of my code
[x] I have added label to the PR (major, minor or patch)
[ ] If new component: Is story for component created in stories-folder?
[ ] If new component: Is tsx-file import added to packages/react/index.tsx?
[ ] If new component: Is css-file added to packages/css/index.css?

github-actions[bot] commented 7 months ago

Please set a versioning label of either major, minor, or patch to the pull request.

aurorascharff commented 7 months ago

@thomaslarsson jeg tror egentlig det bare er git som sliter litt med å vise at det som har skjedd er at hele komponenten har blitt wrappet med forwardref. Er på mobilen så litt vanskelig å si, Kaja kan sikkert verifisere.

kajsaeggum commented 7 months ago

Ser bra ut.

Antar det hovedsakelig er ESlint som står bak de fleste changes her? I fremtiden må du gjerne gjøre format fixes og code fixes i separate commits. Det hadde gjort det enklere for meg å vurdere hva som konkret er endret av deg vs opprydding. :)

Jeg har ikke lintet noe spes, det er som @aurorascharff sier bare Github som sliter litt med comparen i Pr-viewet sitt... Enig i at det gjør det veldig vanskelig å vurdere endringer, men jeg vet ikke helt hvordan man skal kunne unngå det. Denne samme filendringen ser slik ut i VSCode til sammenligning, merk i høyremarginen hvor lite av filen den mener er endret :)

Ser forresten ut til å hjelpe en god del hvis man velger Hide whitespace i pr-viewet: