Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address 2px height difference between WooPay and Google Pay #9137

Closed
bborman22 opened this issue Jul 19, 2024 · 4 comments · Fixed by #9258
Closed

Address 2px height difference between WooPay and Google Pay #9137

bborman22 opened this issue Jul 19, 2024 · 4 comments · Fixed by #9258
Assignees
Labels
focus: checkout payments priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.

Comments

@bborman22
Copy link
Contributor

bborman22 commented Jul 19, 2024

Describe the bug

The bug description is pulled from this comment.

For some reason the WooPay button at 55px is smaller than the Google Pay ECE button at 55px???

Screenshot 2024-07-19 at 10 41 02 AM

Both are 55px according to the dev tools. Maybe the ECE buttons don't take the border into account so a 1px border means the end result is a height of 57px? That seems to be right:

Screenshot 2024-07-19 at 10 41 07 AM

Setting the max-height of the container to 55px results in an obvious cut-off:

Screenshot 2024-07-19 at 10 41 12 AM

It's only just barely noticable when they're side-by-side.

This issue only applies when the Light or Outline theme is chosen for the express checkout settings. The easiest solution is to adjust the WooPay button height to 57px when Light or Outline theme is selected, but it might be worth following up with Stripe to confirm this is the intended behavior.

To Reproduce

  1. Enable both WooPay and Google Pay / Apple Pay.
  2. From the express checkout settings choose the Large (55px) option for button size.
  3. From the express checkout settings choose the Light or Outline theme for buttons.
  4. Load the checkout blocks page in Chrome (so there is only two buttons).
  5. Confirm that there is a 2px height difference between the buttons.

Actual behavior

The button heights are different by 2px with Google Pay being larger than WooPay.

Screenshots

Expected behavior

The button heights should be the same in all themes and button sizes.

Additional context

@bborman22 bborman22 added type: bug The issue is a confirmed bug. focus: checkout payments labels Jul 19, 2024
@pierorocca pierorocca added priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability and removed priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. labels Jul 25, 2024
@bborman22
Copy link
Contributor Author

@reykjalin
Copy link
Contributor

Hmmm, maybe we can fix this somehow with the box-sizing CSS rule?

@asumaran
Copy link
Contributor

asumaran commented Aug 12, 2024

Screenshot 2024-08-12 at 2 50 33 PM

It's seems Stripe is hardcoding the element height to 48px for Google Pay only.

Edit: This seems correct as the button size I'm using is 48px

@asumaran
Copy link
Contributor

I'm going to pick this one and fix it as part of #9169

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: checkout payments priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants