search

NVE / Designsystem

MIT License
1 stars 0 forks source link

PR: lagde message--card komponent #198

Closed amish1188 closed 4 weeks ago

amish1188 commented 1 month ago

Message-card ligner på alert men det er ikke det samme. message-card skal brukes som en lengre boks med teksten for å gi brukere ekstra info i skjemaer f.eks. Jeg lagde den fordi vi skal bruke den i ds dokumentasjonen som tips, varsel bokser.

CodeExamplePreview.vue ble endret til å bruke iife for å kjøre scripter. Grunnen er at når man kjører en script i global scopet lagrer variabler, og når man går til en annen side og kommer tilbake den klager at variabler allerede ble definert. Iife gjør at scopet er lokalt, og den blir ødelagt når vi bytter siden (du kan sjekke problemet i nve-alert f.eks der vi bruker en script for å lukke alerten).

ComponentMethods begrenser nå visning av private og lit sine lifecycle metoder. Vi trenger ikke å vise dem.

Message-card lukk knappen gjør det automatisk at korter blir fjernet fra DOMen. Vi trenger ikke å beholde en gjemt komponent tenker jeg. Hvis vi mener at vi trenger å beholde den, vi kan alltid endre det. Samme anbefaler jeg å gjøre med alerten. Alert egentlig trenger å bli skrevet om til vår egen komponent uten å bruke shoelace. Grunnen er at teksten og tittelen brukes som :after og :before i cssen. Det gjør at man f.eks ikke kan selecte teksten i alerten og kopiere den. Alert komponent er veldig enkelt, og den vil ikke trenge så mye arbeid.

Oppdaterte også et par sider, nå brukes nve-message-card i stedet for Card noen steder, også lagde ny variant på kode preview at den viser kun kode der det trengs (i stepper dok sida).

github-actions[bot] commented 1 month ago

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

github-actions[bot] commented 1 month ago

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-198.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-198.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-198.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-198.westeurope.5.azurestaticapps.net