miljodir / md-components

Design-komponenter for Miljødirektoratet (CSS og React)
https://miljodir.github.io/md-components
MIT License
3 stars 2 forks source link

Add tertiary-theme to MdButton #126

Closed aurorascharff closed 4 months ago

aurorascharff commented 4 months ago

Describe your changes

Implement tertiary button. Design has been approved and added to the design system Figma by our projects' designer.

Animation

Checklist before requesting a review

github-actions[bot] commented 4 months ago

Please set a versioning label of either major, minor, or patch to the pull request.

aurorascharff commented 4 months ago

Approved. If possible, change the wrapper bg color in the demo. It is slightly confusing that the button looks like a "link style button" on white backgrounds right now. (Like in the attached video) This will make it more obvious that the button is white instead of transparent. (Link style)

Det er dette som har blitt bestemt av designer. Det er visst en vanlig funksjonalitet. Selvom den ser ut som en "link". Men den er altså ikke det. Edit: Her var det snakk om demoen/storybook. Edit: Snakket med Thomas og knappen skulle være gjennomsiktig, ikke hvit!

Jeg har tenkt litt på at vi ikke tillater <a / <button differansiering i button, sånn som jeg ahr lagt til i MdLink og MdTile. Altså at hvis man bruker href så blir elementet en a-tag. Men det får vi ta som neste steg etter denne er blitt en vanlig knapp.