Olivero's elements are inheriting the font from the user agent stylesheet. They should be inheriting the font from the the theme.

This can be fixed by adding a font-family: inherit; to the CSS codebase. We need to make sure to run yarn lint:css to make sure the declaration is in the correct order.

CommentFileSizeAuthor
#2 3224466_2.patch693 bytesandy-blum
button-font.png397.8 KBmherchel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mherchel created an issue. See original summary.

andy-blum’s picture

andy-blum
he/him/his
Primary language English
Location Ohio, USA
CreditAttribution: andy-blum at Lullabot commented
FileSize
3224466_2.patch693 bytes

Patch attached

mherchel’s picture

mherchel
he/him
Primary language English
Location Gainesville, FL, US
CreditAttribution: mherchel at Lullabot commented
Status: Active » Reviewed & tested by the community

Looks good (pending tests)!

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented

Looks good 👍 This can be committed after the freeze.

  • lauriii committed df65e9f on 9.3.x 
    Issue #3224466 by andy-blum, mherchel: Olivero: <button> elements are...

  • lauriii committed 61e0890 on 9.2.x 
    Issue #3224466 by andy-blum, mherchel: Olivero: <button> elements are...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Reviewed & tested by the community » Fixed

Committed df65e9f and pushed to 9.3.x. Also cherry-picked to 9.2.x because Olivero is experimental. Thanks!

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Version: 9.3.x-dev » 9.2.x-dev

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.