search

NVE / Designsystem

MIT License
1 stars 0 forks source link

Laget en nve-tag. Kan vise enkel info, og kan ha en lukke-knapp #213

Closed tomapedersen closed 3 weeks ago

tomapedersen commented 3 weeks ago

Erstatter tag-chips og filter-chips som var i skissene tidligere, de ble slått sammen til denne som har mulighet til å ha en lukke-knapp på seg.

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

tomapedersen commented 3 weeks ago

@knutnve : Ser nå at Figma bruker "tag", ikke "chip". Siden tag er et ord i html så foreslår jeg at vi heller døper om komponenten i Figma

tomapedersen commented 3 weeks ago

Gjort en del endringer nå. Men ikke endret emphasis eller extra-text attribute/slot; se kommentarer

@gruble @amish1188

knutnve commented 3 weeks ago

@knutnve : Ser nå at Figma bruker "tag", ikke "chip". Siden tag er et ord i html så foreslår jeg at vi heller døper om komponenten i Figma

Den heter tag i figma og i shoelace. Om det kommer det i konflikt med tag i html, det kan ikk jeg ta stilling til. Så det må dere bestemme. Da må vi endre navn.

amish1188 commented 3 weeks ago

bare en liten info/tips: Den skal ikke komme i konflikt med html tag fordi vi bruker nve-tag

tomapedersen commented 3 weeks ago

@amish1188 , ja konflikten er ikke kodemessig, men "uklarhet om hva vi snakker om"-konflikt. nve-tag kan forstås som "tag i nve-design-systemet", for eksempel <nve-button> er en "tag med nve-prefiks", eller en "nve-tag"

amish1188 commented 3 weeks ago

@tomapedersen skjønner. Jeg syns fortsatt det ikke er så problematisk å kalle komponenten som nve-tag (vi bruker tag begrepet nesten aldri tenker jeg :P), men samtidig har jeg ikke noe mening om hvordan komponenten skal hete. Både tag og chip fungerer for meg. Jeg vil tro at @knutnve tenkte å kalle den tag siden det allerede finnes en komponent i shoelace, men siden du lagde komponent uten å arve fra shoelace, spiller ingen rolle hvordan den heter. Bare så at vi har lik navn både her og i figma. Vil tro det er enklere å bytte navn i figma en i kode (til nve-tag) :P

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

github-actions[bot] commented 3 weeks ago

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

tomapedersen commented 3 weeks ago

Døpt om til nve-tag