Textarea: Teller flyttet ut #2483
Conversation
🦋 Changeset detectedLatest commit: ac48de5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
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 |
Storybook demoEndringer til review: 38c762adcc | 61 komponenter | 398 stories |
| size: TextareaProps["size"]; | ||
| i18n: TextareaProps["i18n"]; | ||
| } | ||
| const Counter = ({ maxLength, currentLength, size, i18n }: CounterProps) => { |
There was a problem hiding this comment.
Counter kan sikkert flyttes ut i egen fil 📁
|
Husker ikke i hvor lett/om det kan løses, men har det vært mulig å legge på border-radius rundt scrollbar her? (Egentlig en egen task, så kan ignoreres for denne PR-en hvis det ikke er en rask fix!) 
Vært nice med en demo av counter + error under "Error"-stories |
|
Oppdatert Error-story. |
|
Testet wrappe-metoden, men fikk ikke wrapperen til å følge bredden til textarea når man resizer den smalere. Må eventuelt løses med JS, og da tenker jeg at det kanskje ikke er verdt kompleksiteten. |
(Ekstrahert fra PR #2457 som nå bare tar for seg autoScrollbar.)
https://github.com/navikt/team-aksel/issues/332
#2326
#2224
#2341
Endringer:
aria-livetilrole=statuspå telleren av semantiske grunner. Ser ut til å funke like bra.resize.