Buttons met en zonder DigiD logo hebben verschillende hoogte

frameless / strapi

https://cms.frameless.io/

4 stars 0 forks source link

Buttons met en zonder DigiD logo hebben verschillende hoogte #541

Open Ollie-nl opened 6 months ago

Ollie-nl commented 6 months ago

Op productie Als bezoeker van een product pagina verwacht deze bezoeker dat alle CTA buttons dezelfde hoogte hebben, 44px (NL DS, Figma), omdat ze dezelfde functies hebben. Buttons die voorzien zijn van een DigiD logo hebben een andere hoogte.

Yolijn commented 2 months ago

@Ollie-nl stel je voor om de DigiD-button dus ook 44px te maken? Dan wordt het logo dus ook 44px hoog.

Robbert commented 2 months ago

Dat komt omdat die 50px hardcoded is: https://github.com/nl-design-system/utrecht/blob/main/components/digid-button/src/index.scss#L7-L9

Dat moet weggehaald worden, of aangepast worden naar var(--utrecht-pointer-target-min-size, 44px)

scar055 commented 1 month ago

oplossing dat Robbert heeft voor gesteld https://github.com/nl-design-system/utrecht/pull/2445

tenzij het anders moet staan