jokul icon indicating copy to clipboard operation
jokul copied to clipboard

Komponent/Mønster: Menu + MenuItem

Open onordbo opened this issue 2 years ago • 5 comments

Det er flere team som per nå har implementert hver sin variant av same type komponent. Toppmeny med tilhørende lenker. Tidligere har dette vært en komponent i Figma, men såvidt meg kjent har det ikke eksistert en ekvivalent i kodebasen. Nå ønsker vi å reintrodusere dette på tvers av designsystemet.

Vi starter med en enkel variant i første versjon, ett enkelt menyelement sammen med en toggle for å skjule/vise menyen på smale skjermer. På sikt kan mønsteret utvides i det uendelige med tillegg for kontekstuelle menyer, badges, statusmeldinger og liknende.

image

🔗 Praktiske lenker (om tilgjengelig)

Designskisser: Patterns → feat: navigation/top-menu Pull request: Link til PR, ved endringer på kode Forhåndsvisning: Preview-link fra @fremtind-bot (om tilgjengelig) Discussion: Link til discussion (om tilgjengelig)

🎯 Sjekkliste

  • [x] Du har laget en branch i Figma og bedt om feedback
  • [ ] Du har sjekket at alt av kode og tester kjører som det skal
  • [ ] Dere har sammen sjekket at resultatet av design og kode matcher

Hvis du står fast og trenger tips kan du ta en kikk i huskelista vår, eller tagge noen fra Jøkul-teamet i dette issuet.

onordbo avatar Mar 01 '23 12:03 onordbo

Denne er mer eller mindre klar for utvikling. (cc @Murstam)

onordbo avatar Mar 21 '23 09:03 onordbo

Er det en idé å supplere et mønster for selve lenkelisten på smale skjermer? Har sett et par versjoner av denne menyen hvor mobilversjonen er en liste med NavLink elementer. Er det ikke litt inkonsekvent å bytte komponent sånn uten videre på tvers av brekkpunkter?

onordbo avatar Mar 21 '23 09:03 onordbo

Skal det skje noe ved klikk på et Menu Item ut av boksen, eller tar vi ikke stilling til det nå @onordbo?

wkillerud avatar Mar 21 '23 12:03 wkillerud

Nice. Vi bruker ikke den komponenten, så blir nok ikke vårt team som utvikler hvis du tenkte det.

Murstam avatar Mar 21 '23 12:03 Murstam

Var @ivnai som savnet den.

Murstam avatar Mar 21 '23 12:03 Murstam