WIP Byttet gamle datovelgeren med ny fra designsystemet #1056
Conversation
|
|
Please retry analysis of this Pull-Request directly on SonarCloud |
| }; | ||
|
|
||
| return ( | ||
| <DatePicker {...datepickerProps}> |
There was a problem hiding this comment.
Her kan vi også leggje til fromDate (lenke til props) på DatePicker slik at brukaren ikkje kan velje ein dato i fortida i kalendermodalen.
| <FormikDatoVelger name={`arbeidsliste[${index}].frist`} /> | ||
| <FormikDatoVelger | ||
| name={`arbeidsliste[${index}].frist`} | ||
| label={`arbeidsliste[${index}].frist`} |
There was a problem hiding this comment.
| label={`arbeidsliste[${index}].frist`} | |
| label="Frist" |
Label er teksten vi viser ut til brukar. Eg veit denne datovelgaren er i arbeidslista og ikkje synleg for brukar, men trur det er lurt å fikse det likevel.
| <FormikDatoVelger name="frist" /> | ||
| <FormikDatoVelger | ||
| name="frist" | ||
| label="frist" |
There was a problem hiding this comment.
| label="frist" | |
| label="Frist" |
| const inputDato = moment(input); | ||
| const fraDato = moment(); | ||
| if (input && !erGyldigISODato(input)) { | ||
| error = 'Ugyldig dato'; |
There was a problem hiding this comment.
| error = 'Ugyldig dato'; | |
| error = 'Ugyldig dato: datoen skal være på formatet dd.mm.åååå'; |
| import classNames from 'classnames'; | ||
| import 'nav-datovelger/lib/styles/main.css'; | ||
| import SkjemaelementFeilmelding from 'nav-frontend-skjema/lib/skjemaelement-feilmelding'; | ||
| import {Label} from '@navikt/ds-react'; | ||
| import {DateInputProps, DatePicker, ErrorMessage, Label, useDatepicker} from '@navikt/ds-react'; |
There was a problem hiding this comment.
Ubrukt import gjer at vi ikkje får deploya greina
| import {DateInputProps, DatePicker, ErrorMessage, Label, useDatepicker} from '@navikt/ds-react'; | |
| import {DateInputProps, DatePicker, ErrorMessage, useDatepicker} from '@navikt/ds-react'; |
| return safeFormat(moment(dato || undefined), 'YYYY-MM-DD'); | ||
| } | ||
|
|
||
| export const validerFristFelt = (input: string): string | undefined => { |
There was a problem hiding this comment.
Eg trur ikkje denne plukkar opp alle ugyldige datoar brukaren skriv inn. Kanskje vi kan bruke onValidate fra useDatePicker i tillegg?
Ting som slepp gjennom sjekken no, og som brukaren ikkje får lagra:
- 2024-12-21 (yyyy-mm-dd)
- 99.99.9999 (rett format, ugyldige datoar)
- acorkcho (ikkje gyldig dato)
Vi kan også seie at det er godt nok for no, og gjere fullstendig validering seinare. Det som er gjort her er mykje betre enn det vi har i visittkort no.
| dayPickerProps={{ | ||
| className: 'datovelger__DayPicker' | ||
| }} | ||
| <DatoVelger |
There was a problem hiding this comment.
Kanskje vi kan flytte label="Frist", size="small" og validate={validerFristFelt} inn hit, sidan dei er like kvar gong FormikDatoVelger vert brukt?
| defaultSelected: field.value ? new Date(field.value) : undefined, | ||
| inputFormat: 'dd.MM.yyyy', | ||
| onDateChange: (date?: Date) => { | ||
| setFieldValue(field.name, date ? toReversedDateStr(date) : null); |
There was a problem hiding this comment.
Det ser ut som vi ikkje lenger sender med frist: null til backend når inputfeltet er tomt. Det gjer at vi ikkje kan slette fristar frå huskelappar om ein har lagra ein frist tidlegare.
Eg veit ikkje om det er her feilen er, men vi må få fiksa det før vi tek arbeidet ut i master.
| }} | ||
| <DatoVelger | ||
| formikProps={formProps} | ||
| ariaLabel={ariaLabel} |
There was a problem hiding this comment.
Treng vi å bruke aria-label her, eller er det nok å la inputen få namn til skjermlesar frå det tilhøyrande<label>-elementet?
No description provided.