fremtind / jokul

Jøkul er et designsystem utviklet av Fremtind.
https://jokul.fremtind.no
MIT License
63 stars 33 forks source link

Lenkeliste-komponent fra Fremtind.no inn i Jøkul #3920

Closed olejorgenbakken closed 1 week ago

olejorgenbakken commented 1 month ago

Vi har en del lenker til andre løsninger i dag, og bruker cards fordi de står veldig ut på siden vår. Problemet vi fort får er at alt på siden vår er cards, så det hadde vært nice med et annet virkemiddel som feks denne lenkeliste-komponenten:

Screenshot 2024-07-30 at 14 51 11
olejorgenbakken commented 1 month ago

Denne varianten hadde også vært sykt nice, fordi vi begynner å få noen lengre sider vi skulle vist en innholdsfortegnelse på.

Screenshot 2024-07-30 at 14 57 39
ivarni commented 1 month ago

Tanker om hvordan APIet burde se ut?

I mitt hode er kanskje den første en uordnet liste og den andre en ordnet? Da ligner det litt på <ul> og <ol> som de fleste sikkert er vant til å forholde seg til.

Kunne da sett ut som feks

<UnorderedLinkList>
    <LinkItem><a href="" /></LinkItem>
    <LinkItem><a href="" /></LinkItem>
</UnorderedLinkList>

og

<OrderedLinkList>
    <LinkItem><a href="" /></LinkItem>
    <LinkItem><a href="" /></LinkItem>
</OrderedLinkList>

feks noe som dette

Så kan man eventuelt ha en prop for om lenka er ekstern eller ikke (har pil vs ikke har pil). Kan jo og bare si at children må være Link eller NavLink komponentene vi allerede har.

olejorgenbakken commented 1 month ago

Jeg syns dette høres veldig fornuftig ut!

I tillegg tror jeg dokumentasjonen da gjerne kunne sagt noe om at hvis man bruker <ol>-versjonen til innholdsfortegnelse burde den wrappes i <nav>. Jeg fant ut av dette først nå selv, mens jeg leste dokumentasjonen til <nav> for en oppgave jeg satt med haha.

ivarni commented 1 month ago

Ah, det kunne vi nok gjort på fremtind.no også men det er jo naturlig at vi bikkjemater komponenten der hvis den finnes i jøkul så vi kan nok ordne det da.

Er dette kanskje en god oppgave for onboarding @piofinn ?