IS-3017: Tydeliggjøre søket #574
Conversation
| @@ -78,7 +124,7 @@ export default function SokPerson() { | |||
| }; | |||
|
|
|||
| const isValidInitials = (initials: string): boolean => { | |||
| return initials === '' || (initials.length <= 3 && initials.length > 1); | |||
| return initials === '' || (initials.length <= 4 && initials.length > 1); | |||
There was a problem hiding this comment.
Er det en juridisk grunn til at vi har begrenset det til ett mellomnavn? Kunne vi evt utvidet til fire initialer?
There was a problem hiding this comment.
Vet ikke 🤔 Litt usikker på om vi har begrensninger i backend eller om det funker med n antall initialer i backend?
There was a problem hiding this comment.
Vi har ingen øvre grense i backend, bare at det skal være minst to. Kan ikke se noen grunn til å ikke kunne utvide til fire...
| <BodyShort>{texts.info}</BodyShort> | ||
| <HStack gap="8" align="end"> | ||
| <BodyLong>{texts.info}</BodyLong> | ||
| <div className="grid grid-cols-[auto,auto,auto] gap-x-8 gap-y-2 align-end max-w-max"> |
There was a problem hiding this comment.
Dette ser kanskje litt voldsomt ut, men det er egt bare for å få error meldingene under det respektive input feltet med grid istedenfor flex.
src/sider/sokperson/SokPerson.tsx
Outdated
| initials: { | ||
| title: 'Hvordan fyller jeg inn initialer?', | ||
| text: | ||
| 'For å optimalisere søket kan du fylle inn initialene til den du vil søke opp. Feltet krever forbokstaven for fornavn og etternavn uten punktum mellom bokstavene. Det er mulig å legge til forbokstavene for mellomnavn for mer nøyaktig søkeresultat. Rekkefølgen på fornavn, mellomnavn og etternavn i søket er viktig.', |
There was a problem hiding this comment.
Stemmer det at man ikke kan skrive med punktum mellom bokstavene? Jeg kan ikke se at vi har noen feilmeldinger for det i front-end. Eller er det kanskje ikke reelt å prøve å skrive inn "I.N.G" istedenfor "ING"?
ingring
force-pushed
the
IS-3017-tydeliggjore-sok
branch
from
9633cd4 to
05cdbf2
Compare
src/sider/sokperson/SokPerson.tsx
Outdated
| <> | ||
| Her kan du fylle inn fødselsdato til den du vil søke opp. For å kunne | ||
| søke må sifrene fylles inn på formatet: | ||
| <strong> dag, måned, år</strong>, i den rekkefølgen. Ønsker man et mer |
There was a problem hiding this comment.
Blir det rart å si et format med komma, legger det opp til at folk skriver 28, 01, 2025?
Kanskje heller skrive For å kunne søke må sifrene fylles inn i rekkefølgen: <strong> dag måned år</strong> eller noe?
There was a problem hiding this comment.
Godt poeng, kan prøve uten komma!
ingring
force-pushed
the
IS-3017-tydeliggjore-sok
branch
2 times, most recently
from
b2b3cc0 to
37b8a4c
Compare
|
For å tydeliggjøre initial feltet enda mer har jeg nå endret description og error meldingen. Søket ser foresten sånn ut nå, hva tenker vi om det? |
ingring
force-pushed
the
IS-3017-tydeliggjore-sok
branch
from
37b8a4c to
b471a2d
Compare
Hva har blitt lagt til✨🌈
Screenshots 📸✨
Før:
Nå:
Skjermopptak.2025-01-27.kl.15.57.22.mov