navikt / Designsystemet-old

Designsystem-teamet i NAV sitt offisielle område på Github
https://design.nav.no
MIT License
8 stars 1 forks source link

Bør FnrInput ha lengde begrenset til 11 siffer som default? #147

Closed Lillebo closed 4 years ago

Lillebo commented 4 years ago

Ref. Slack-forespørsel

Bør det gjelde både fysisk lengde og antall tegn den aksepterer?

https://design.nav.no/components/fnrinput

erlendev commented 4 years ago

Nei, jeg syns ikke det er lurt å sette den begrensningen. Kan man ikke heller åpne for at teamene kan sette maxLength dersom det gir mening for dem? Tenker det er dumt å fjerne muligheten for å gi beskjed til brukerne om at det man har skrevet inn i feltet er for langt - hvorfor et fødselsnummer er ugyldig er ikke alltid innlysende, og det er nyttig å kunne si om det bruker har tastet inn er for kort, for langt, eller bare er på ugyldig format.

frodehansen2 commented 4 years ago

Enig med Erlend. Og når en først åpner for at komponenten har litt, bør den kanskje også håndtere ulike måter en bruker kan legge inn fnr på? Dette er kanskje noen copy-paster, og da er det kanskje formatert med space. Så det hadde vært fint om komponenten blir robust nok til å takle de mest vanlige måtene fnummeret er formatert på. Sånn sett kunne en kanskje fått info om det gyldige nummeret var fnr eller dnr?

karinaldw commented 4 years ago

Enig i at det kan være lurt å satse på feilmelding istedenfor maks antall tegn. Tror likevel at det er lurt å visuelt ha den på 11 tegn, slik at det gir en indikasjon på hvor lang input som er forventet.

Lillebo commented 4 years ago

Enig med alle 👍 Foreslår at vi legger inn eksplisitt bruk av bredde="S" i eksemplene på komponent-siden + at vi legger inn en liten oppfordring til å bruke dette på komponentens tilgjengelighets-fane, med en peker hit: https://design.nav.no/components/input#lengde

KenAJoh commented 4 years ago

Fikset med navikt/nav-frontend-moduler#711