@pierorocca Does this issue have everything it needs to be ready for someone to work on? I see that there is a draft notice at the beginning of the issue, and that's the reason why I am asking this.

It's not ready, and is pending design guidance. The current behavior is not what design expected so Nikki is taking another look. Likely will have a set of revised font sizes for each button size.

Thank you for the feedback @pierorocca . I can move this issue to the Shopper Experience team, or do you think that there is a another team that could work on this issue?

Assigned it to myself. Thanks @zmaglica.

Great. Thank you for the feedback. Moving this to the Other Teams Queue as part of Gamma Triage process.

I've updated the issue with the correct text sizes and added an additional Figma reference

Hi @nikkivias , in the Figma reference the button height and the logo height are the same. I'm confused how this is supposed to work because if they are the same then you get something like this:

Is that the intention or am I missing something on the design?

@LCmry that does look like a typo. What were the values prior because the only thing that needed updatig relative to the existing implementation were the font sizes.

@pierorocca Button height for default (small) is 40px, 'medium' is 48px while the logo is 27px, for large it's button 56px and logo 32px.

Also, in this issue and the design say "small, default, large" but I think the small is default and it's really more like "default, medium, large" unless we want to change that.

Medium is definitely the current default. Originally 40px was the default for a new merchant and it was deemed too small relative to the size of most theme's Cart and Checkout buttons. 48px size became the new default.

@LCmry I'd keep those logo sizes as is at 27px and 32px. Only the font sizes were part of the problem we were trying to resolve.