wp-calypso icon indicating copy to clipboard operation
wp-calypso copied to clipboard

Replace the icon of the button "close checkout" from `chevron-left` to `cross`

Open southp opened this issue 1 year ago • 7 comments

Related to Automattic/dotcom-forge#7674

Proposed Changes

This PR updates the "close checkout" button icon from chevron-left to cross:

In production In this PR
CleanShot 2024-06-28 at 18 49 46@2x CleanShot 2024-06-27 at 18 57 28@2x

Why are these changes being made?

Using the chevron-left icon may lead to false expectation of what the button does like what we saw in the discussion in Automattic/dotcom-forge#7674. It looks like a back navigation button, while it's actually a close button if we hover on it: CleanShot 2024-06-28 at 18 50 33@2x

The cross icon matches the intended behavior better in this case.

Testing Instructions

Go to the checkout screen and confirm that the icon has been updated accordingly.

Pre-merge Checklist

  • [ ] Has the general commit checklist been followed? (PCYsg-hS-p2)
  • [ ] Have you written new tests for your changes?
  • [ ] Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
  • [ ] Have you checked for TypeScript, React or other console errors?
  • [ ] Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
  • [ ] Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
  • [ ] For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

southp avatar Jun 27 '24 10:06 southp

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

  • notifications
  • wpcom-block-editor

To test WordPress.com changes, run install-plugin.sh $pluginSlug update/change-close-checkout-icon-as-close on your sandbox.

matticbot avatar Jun 27 '24 11:06 matticbot

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

matticbot avatar Jun 27 '24 11:06 matticbot

@Automattic/shilling Could you help me confirm if this change makes sense? I'm wondering if there is any context justifying chevron-left that I'm not aware of :)

southp avatar Jun 28 '24 10:06 southp

@Automattic/shilling Could you help me confirm if this change makes sense? I'm wondering if there is any context justifying chevron-left that I'm not aware of :)

I'm not sure of any historical context here other than "Jetpack and Akismet will not have a back button at all, but dotcom will."

IMO, I think the chevron looks more correct as it suggests directionality whereas a cross suggests "closing a modal or window" which isn't really the case here?

jjchrisdiehl avatar Jun 28 '24 19:06 jjchrisdiehl

The change from an "x" to the chevron was made as part of this design refresh: pbOQVh-3pj-p2

I believe @JanaMW27 requested the change, but it could have just been part of the alignment with other Calypso screens. I don't have any objection to this change back to an "x".

michaeldcain avatar Jun 28 '24 20:06 michaeldcain

@jjchrisdiehl Thanks for pitching in. It's very helpful to know that Jetpack and Akismet doesn't have that button at all :)

IMO, I think the chevron looks more correct as it suggests directionality whereas a cross suggests "closing a modal or window" which isn't really the case here?

The issue here is that it actually doesn't work as back naivgation, but as closing the checkout page like its tooltip suggested. On wp.com, the ideal behavior is to proceed to the next step as I outlined in this comment. For example, on the main onboarding flow, users go through: the user step -> the domain step -> the plans step -> (checkout if there is a paid item in the cart) -> launch pad. When they click that button, they should arrive the launch pad. Thus, it's great to know that by updating the icon here Jetpack and Akismet likely won't be affected, since I'm better aware of the wp.com side of things.

Does that make sense to you?

southp avatar Jul 01 '24 02:07 southp