Kilden til status på komponentene ligger nå i Komponentoversikt.md

gruble commented 4 weeks ago

Hensikten med denne PR'en er å kunne vedlikeholde status på komponenter på dokumentasjons-nettstedet (i stedet for i Figma).

Jeg prøvde først å lage en enkel tabell i en markdown-fil med navn på hver komponent + status i Figma og kode. Dette funka, men hvis vi skulle ha en fin formatering av statuser med farger og linker til Figma, ble markdown-fila mye lengre og vanskeligere å finne fram i. Derfor gikk jeg tilbake til å generere sida slik Joel allerede hadde, for å få skilt presentasjon og data på en ryddig måte. Men jeg har flyttet dataene til selve markdown-fila, for å gjøre det lettere å redigere dem.

Har gjort noen endringer i komponent-oversikten:

denne henter nå sine data fra Komponentoversikt.md
For å legge til komponenter eller endre status på en komponent, rediger lista som sendes inn til ComponentOverview i Komponentoversikt.md. Dette kan gjøres direkte fra sida ved å klikke på Rediger denne sida i Github (men ikke før denne PR'en evt. fullføres)
Har forenklet strukturen på det vi trenger for å vise lista, sammenlignet med componentVersionsData.js, for å gjøre det lettere å vedlikeholde
Fjernet døde linker. Kun komponenter vi har laget, har link nå
Jeg har fjernet statusen Mangler link i Figma og heller latt være å vise link-ikonet når vi mangler en link
Lista er nå sortert alfabetisk på komponent-navn, så det stemmer med rekkefølgen i menyen under 'Komponenter'.
Beskrivelse på komponenten vises foreløpig ikke. Vi kan ta inn dette ved behov
Det var en del komponenter som manglet. Jeg har lagt inn disse.
La også inn en oppsummering av antall komponenter nederst, fordi jeg var nysgjerrig på hvor langt vi har kommet

Gjenstår:

[ ] NB! En del av komponentene hadde feil status, eller tvilsom status i kode. Jeg har fjernet status for disse, så dette bør sjekkes og oppdateres
[ ] Hadde vært kjekt å kunne sortere på hver av status-kolonnene også
[ ] Kunne ønsket meg noe lavere rad-høyde

github-actions[bot] commented 4 weeks ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-205.westeurope.5.azurestaticapps.net

github-actions[bot] commented 4 weeks ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-205.westeurope.5.azurestaticapps.net

github-actions[bot] commented 4 weeks ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-205.westeurope.5.azurestaticapps.net

github-actions[bot] commented 4 weeks ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-205.westeurope.5.azurestaticapps.net

NVEJoel commented 3 weeks ago

Ser bra ut! Snakkat med Knut om upplægget og han synes det også er bra. To ønskemål:

En til kolumn som heter "Git issue" dær tanken er att designerna kan lægga in Github issue lænken dær man beskriver hva som må gøras eller ændras.
Kan man få de olika status i raden Totalt antall komponenter på rad istællet så det blir tydligare?
Vore snyggt med en Søk funktion men det kan vi lægga til senare.

knutnve commented 3 weeks ago

Innspill:

Skal vi fjerne nve- foran komponentnavn for å gjøre tabellen enda mer lesbar og oversiktlig?
Kunne vi lagt til git hub logo eller figma logo dersom det er link til de?
Om vi har en kommentar som burde være synlig, kan vi legge til rette for det også

Typ sånn?

gruble commented 3 weeks ago

Innspill:

Skal vi fjerne nve- foran komponentnavn for å gjøre tabellen enda mer lesbar og oversiktlig?

Vi velger å ikke fjerne nve-foran komponentnavn foreløpig fordi det krever at vi må ha et ekstra navnefelt på komponenten. Kan legges til seinere hvis vi ønsker ha det.

Kunne vi lagt til git hub logo eller figma logo dersom det er link til de?

Github-logo trenger vi nok ikke. Tror ikke det er så relevant med link til kildekoden til komponenten. Men vi har allerede link til brukerveiledninga for komponenten. Figma-logo har vi. Dessverre fant jeg ikke sort-hvitt-svg'en for logoen i Figma, så beholder den vi hadde i farger.

Om vi har en kommentar som burde være synlig, kan vi legge til rette for det også

Venter litt med denne, men tar det inn hvis vi får til å oppdatere status manuelt. Det er lett å legge inn seinere.

gruble commented 3 weeks ago

Ser bra ut! Snakkat med Knut om upplægget og han synes det også er bra. To ønskemål:

En til kolumn som heter "Git issue" dær tanken er att designerna kan lægga in Github issue lænken dær man beskriver hva som må gøras eller ændras.

Kan man få de olika status i raden Totalt antall komponenter på rad istællet så det blir tydligare?

Vore snyggt med en Søk funktion men det kan vi lægga til senare.

Fikset linjeskift mellom hver status. Jeg har laget en egen sak på det med å linke inn issues (#220).

github-actions[bot] commented 3 weeks ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-205.westeurope.5.azurestaticapps.net

NVE / Designsystem

Kilden til status på komponentene ligger nå i Komponentoversikt.md #205