Automattic / woocommerce-payments

Accept payments via credit card. Manage transactions within WordPress.
https://wordpress.org/plugins/woocommerce-payments/
Other
171 stars 69 forks source link

Updates to payment icons on Payments Connect screen #8567

Closed elizaan36 closed 4 months ago

elizaan36 commented 4 months ago

Describe the bug

Two updates to the payment icons on the Payments Connect screen. ✅ Remove duplicate WooPay logo [Will be fixed in another issue]

Small note - Can we remove the period after "& more"? It's not necessary as it's not a complete sentence.

To Reproduce

  1. Create a new store with WooPayments installed and activated
  2. Go to the Payments Connect screen
  3. See payment icons

Expected behavior

Here's a mockup of the correct icon layout.

image

cc @anu-rock

csmcneill commented 4 months ago

The duplicate WooPay icon was reported in #8505 and may be resolved by #8521

elizaan36 commented 4 months ago

I updated the original comment to mention that the WooPay icon is being fixed and added the replacement for the Afterpay icon. I'm noting here that there are two additional placements of the payment icons that will need to be updated in WooCommerce core, the Set up payments task, and the Payments Settings banner.

Screenshot 2024-04-03 at 8 53 11 PM Screenshot 2024-04-03 at 12 26 53 PM

@nikkivias and I are discussing a shared component for the payment icons that can stay consistent throughout all placements, mobile/ desktop versions, and only needs to be updated in one place. @pierorocca Would that work land with your teams or with Moltres, so we know who to collaborate with on this.

pierorocca commented 4 months ago

A couple more callouts. The Discover and Diners logos are outdated. Attached are samples from Discover's brand guidelines.

Downloadable assets - https://discover.widen.net/s/rtwc682nvd/discover-global-network-acceptance-mark-2 https://www.discoversignage.com/free-signage-logos/

Preview- image

csmcneill commented 4 months ago

@pierorocca @elizaan36 Since this PR is closed, I opened #8580 to tackle the Discover/Diners changes :)