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

[FEATURE] Timeline interne-flater #1665

Merged
merged 176 commits into from
Jan 20, 2023
Merged

[FEATURE] Timeline interne-flater #1665

merged 176 commits into from
Jan 20, 2023

Conversation

KenAJoh
Copy link
Collaborator

@KenAJoh KenAJoh commented Oct 14, 2022

WIP branch for ny tidslinje bruk på interne flater

Tidligere brukt kode

Storybook:

Forslag zoom

<TimeLine.Zoom>
  <Timeline.Zoom.Item label="3 mnd" interval="month" count={3} />
  <Timeline.Zoom.Item label="7 mnd" interval="month" count={7} />
  <Timeline.Zoom.Item label="9 mnd" interval="month" count={9} />
</TimeLine.Zoom>

@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2022

Chromatic build-output:

commit: 649f70a8b

@KenAJoh
Copy link
Collaborator Author

KenAJoh commented Oct 18, 2022

Forslag til førsteutkast for API

<Timeline>
  <Timeline.Row label=ReactNode>
	    <Timeline.Period from=Date to=Date .../>
	    <Timeline.Period from=Date to=Date icon=ReactNode onClick >
		      Statusboble-innhold
	    </Timeline.Period>
  </Timeline.Row>
  <Timeline.Pin date=Date>
	    Statusboble-innhold
  </Timeline.Pin>
  <Timeline.Pin date=Date>
	    Statusboble-innhold
  </Timeline.Pin>
</Timeline>

  • Bør alle rows være i en egen wrapper eks Timeline.Rows?
  • Bør row ha en egen komponent for label?
<Timeline.Row>
	<Timeline.Label>
	<Timeline.Periods>
		<Timeline.Period>

@KenAJoh KenAJoh assigned kschieren and unassigned larseirikhansen Oct 18, 2022
@kschieren
Copy link
Contributor

Det vil vel kun være en gruppering av Rows per tidslinje, så jeg tenker at det å tilby Timeline.Rows er å gi brukeren et unødvendig ansvar da dette er noe vi kan bygge inn fra vår side. Med mindre du ser noen fordeler ved å gi brukeren kontroll over dette? @KenAJoh

@KenAJoh
Copy link
Collaborator Author

KenAJoh commented Oct 18, 2022

Det vil vel kun være en gruppering av Rows per tidslinje, så jeg tenker at det å tilby Timeline.Rows er å gi brukeren et unødvendig ansvar da dette er noe vi kan bygge inn fra vår side. Med mindre du ser noen fordeler ved å gi brukeren kontroll over dette? @KenAJoh

Tenker egentlig også det for denne versjonen. Hvis vi frem i tid hvis vi skal tilby "accordion-rows", er wrapper nødvendig for at det ikke skal være breaking da? Kanskje default er uten wrapper, men med collapsible (hvis vi implementerer i fremtiden) at man har en "Timeline.CollapsibleRow" eller noe lignende.

@kschieren
Copy link
Contributor

Det vil vel kun være en gruppering av Rows per tidslinje, så jeg tenker at det å tilby Timeline.Rows er å gi brukeren et unødvendig ansvar da dette er noe vi kan bygge inn fra vår side. Med mindre du ser noen fordeler ved å gi brukeren kontroll over dette? @KenAJoh

Tenker egentlig også det for denne versjonen. Hvis vi frem i tid hvis vi skal tilby "accordion-rows", er wrapper nødvendig for at det ikke skal være breaking da? Kanskje default er uten wrapper, men med collapsible (hvis vi implementerer i fremtiden) at man har en "Timeline.CollapsibleRow" eller noe lignende.

Ja det høres fornuftig ut i mitt hode i hvert fall, så da blir det en wrapper rundt collapsible rows eventuelt

@KenAJoh
Copy link
Collaborator Author

KenAJoh commented Oct 24, 2022

Kort oppsummering fra UU/UX møte:

Aria-hidden på axis header

Teste å løse det på samme måte som Stepper:

  • ol -> li
  • aria-labelledby header (row-header)

Date-format:

  • suksess fra 01.01.2020 til 01.02.2020

Pins:

  • Bake inn pins i hver rad (fokuserbar)?
  • Gi hver pin et navn: eks [pin] for lettere søk

@KenAJoh KenAJoh marked this pull request as ready for review January 20, 2023 11:42
@KenAJoh KenAJoh merged commit 6ab84e5 into master Jan 20, 2023
@KenAJoh KenAJoh deleted the new/tidslinje branch January 20, 2023 12:50
@github-actions github-actions bot mentioned this pull request Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants