search

NVE / Designsystem

MIT License
1 stars 0 forks source link

feat(nve-icon-button): Added new component #209

Closed amish1188 closed 3 weeks ago

amish1188 commented 4 weeks ago

Ny komponent nve-icon-button. Den arver fra nve-button, men kun viser ikonen.

Måtte bruke iconName siden name property allerede eksisterer på knappen, og jeg ville ikke overskrive den

Jeg måtte justere selve nve-button litt fordi det var en del endringer som påvirket hvordan icon knapp vises. F.eks størrelse på ikonen skal alltid være 24px. Før smallhadde ikon med 20px.

Jeg la til transisjon på nve-button også, den burde være 0.3s. Per i dag er det shoelace sin transisjon som varer 0.5ms.

github-actions[bot] commented 4 weeks ago

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

knutnve commented 3 weeks ago

Godkjent her! Må oppdateres i Figma.

Når det kommer til min-width kan vi gjøre det. Skal undersøke hvilken verdi som skal være minimum.

tomapedersen commented 3 weeks ago

Liten kommentar: Bør gjøres mer eksplisitt at man må sette aria-label (eller tilsvarende). En knapp med kun ikon er ikke wcag-vennlig

github-actions[bot] commented 3 weeks ago

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