Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Utvide datePickerDefaultProps med onWeekNumberClick #2285

Closed

Conversation

frodehansen2
Copy link
Contributor

@frodehansen2 frodehansen2 commented Sep 14, 2023

Description

Utvide DatePickerDefaultProps med onWeekNumberClick fra DayPickerBase.

Change summary

Utvide DatePickerDefaultProps med onWeekNumberClick fra DayPickerBase.

Alternativ løsning til denne PR'en - og kanskje det som hadde vært best?

Kanskje det kan være et bedre alternativ å tilby hele DayPicker direkte, men hvor en har lagt til alle NAV tilpasningene på layout, locale, roles etc. Da vil brukere kunne bruke den med all funksjonalitet som den har, men samtidig høre til i NAV-familien.

Lagde egen PR på alternativet:
#2301

@changeset-bot
Copy link

changeset-bot bot commented Sep 14, 2023

🦋 Changeset detected

Latest commit: 704ade4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@navikt/ds-react Patch
@navikt/ds-css Patch
@navikt/ds-tokens Patch
@navikt/ds-tailwind Patch
@navikt/aksel-icons Patch
@navikt/aksel Patch
@navikt/aksel-stylelint Patch

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

@frodehansen2 frodehansen2 changed the title Utvide date picker default props med on week number click Utvide datePickerDefaultProps med onWeekNumberClick Sep 14, 2023
@github-actions
Copy link
Contributor

Storybook demo

0c7e0352c | 57 komponenter | 372 stories

@KenAJoh
Copy link
Collaborator

KenAJoh commented Sep 14, 2023

🙌
Skal få sett på den med resten av teamet snart, må kanskje gjøre noen endringer på hvordan "uke"-knappen vises.

@frodehansen2
Copy link
Contributor Author

frodehansen2 commented Sep 14, 2023

Ja, den bør kanskje skilles visuelt fra selve dagene i uken. Alternativt legge på en strek til høyre for ukene.

@frodehansen2
Copy link
Contributor Author

frodehansen2 commented Sep 15, 2023

Forslag til layout for å skille ut ukenumrene.
image

Poc
https://6503393a90f03568e0629283-dpaqvskhmx.chromatic.com/?path=/docs/component-dayselector--docs

En får ReactDayPicker med samme layout ved å bruke komponenten direkte (<DayPicker.../>, men en må da passe på at klasser settes likt det som kommer fra DatePicker. Ulempen vil være at det vil være skjørt dersom noen av klassene på DatePicker skulle endre seg. Kanskje det vil være et alternativ å tilby ReactDayPicker clean, dvs. kun en NAV-stylet versjon av ReactDayPicker?

@frodehansen2 frodehansen2 deleted the utvide-DatePickerDefaultProps-med-onWeekNumberClick branch September 28, 2023 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants