Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Email: Comparison link should open in new tab #95812

Closed matt-west closed 15 hours ago

matt-west commented 3 days ago

Quick summary

The link to the page comparing email options should open in a new tab and have an external link icon.

Image

Steps to reproduce

  1. Go to WP Admin for a site.
  2. Go to Hosting ➡️ Domains
  3. Add new domain
  4. Select a domain

What you expected to happen

The link should open a new tab and display the external link icon.

What actually happened

Page opens in the current tab.

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

Robertght commented 2 days ago

📌 REPRODUCTION RESULTS

📌 FINDINGS/SCREENSHOTS/VIDEO

WkQoj1.png

📌 ACTIONS

📌 Message to Author @matt-west I'm able to select one of the email services and proceed with my selection. Is this flow currently broken on your end, or should it be different in design?

p-jackson commented 2 days ago

I've got a fix here: https://github.com/Automattic/wp-calypso/pull/95841

We don't usually open Calypso links in a new tab, but an exception is warranted here. When the user is purchasing a domain they're in the middle of a purchase flow, and we would prefer to keep them in that flow.

The same comparison link exists at /email/{{ site slug }} when the site already has a custom domain. In this case the user isn't in the middle of a flow, and so the exception doesn't hold in this situation. So it hasn't been updated to an external link here.

@matt-west do you agree with my reasoning for why an exception is warranted in the "domain purchase" case but not the "existing domain" case?

p-jackson commented 1 day ago

From https://github.com/Automattic/wp-calypso/pull/95841#pullrequestreview-2404179158

The fact that clicking Select on the comparison page takes you back to the purchase flow makes for an odd experience, IMO.

@fredrikekelund brings up a good point. Once you're done comparing and you click Select you're taken back to the domain purchasing flow.

https://github.com/user-attachments/assets/c783f70b-5d59-4d20-ac65-4f6015002cf6

It's sort of strange that clicking "Select" doesn't actually select the option, but instead navigates back to the previous page. But it does keep the user in the purchase flow. And I can see why they have to navigate back to the previous page because they need to fill in the Google Email related fields.

@matt-west will be AFK for a couple days, but I think this calls into question whether opening in a new tab really improves the experience. Because now the user has the same flow open in two tabs.

wojtekn commented 1 day ago

Maybe we should skip this in favor of https://github.com/Automattic/wp-calypso/issues/95864?

matt-west commented 1 day ago

do you agree with my reasoning for why an exception is warranted in the "domain purchase" case but not the "existing domain" case?

This make sense to me @p-jackson.

@fredrikekelund makes a good point about this resulting in users having the same flow open in two tabs though. That could be more confusing.

Ideally we fix this by combining the two pages in #95864. I agree with @wojtekn that we should skip this issue in favour of fixing it properly.

fredrikekelund commented 1 day ago

+1 for closing this in favor of https://github.com/Automattic/wp-calypso/issues/95864

p-jackson commented 15 hours ago

Closing in favour of https://github.com/Automattic/wp-calypso/issues/95864