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
G Suite: Update copy on stats nudge to remove 'click here' language #38856
Conversation
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: App Entrypoints (~98825 bytes added 📈 [gzipped])
Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used. Sections (~279235 bytes removed 📉 [gzipped])
Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. Async-loaded Components (~70317 bytes removed 📉 [gzipped])
React components that are loaded lazily, when a certain part of UI is displayed for the first time. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
@stephanethomas who would be a good person to make a call on this? |
Thanks for the ping @belcherj! @danhauk, @Automattic/larimar is now responsible for our G Suite integration. The change looks good to me, I'll put @fditrapani in the loop just in case. Note that we have plans to revisit the copy of this nudge as it is known to be misleading (see p9j7e4-w1-p2#comment-875). I'm not sure when this will happen though, given our other priorities. |
Tested it, and it works fine . |
So sorry for my laggy response, I was at the marketing division meetup last week and missed the ping! Good call on losing the "click here." I think your update is an improvement and looks fine, and offer two thoughts in case they're useful:
|
I don't know the answer to that question but I think that if we go that route we could also revisit the first part of the heading. As I mentioned earlier, this part is confusing as it implies we know customers are trying to contact the user. Your proposal seems like a clearer alternative to me:
We also refer to this email as professional email in other pages, we may want to reuse the same term here (it's just a suggestion though):
I would be tempted to refine the sentence below as well - to make it clearer that users can actually create other mailboxes than
What do you think @michelleweber? 🙂 |
I probably wouldn't bother with the "professional" here in the heading, but I like it in the text (also -- on the other page where it appears, it shouldn't be capitalized). I like this tweaked version of your copy: Get custom email addresses with [domain] (That said, I'd also be all for getting rid of the problematic "click here" language quickly while we work on larger improvements.) |
e2ce80a
to
c55685f
Compare
e1ac0bb
to
438670b
Compare
{ | ||
<p> | ||
{ translate( | ||
"Let customers reach you at {{strong}}contact@%s{{/strong}}. We've partnered with Google to offer you email, storage, docs, calendars, and more integrated with your site.", | ||
"An email address like {{strong}}contact@%(domain)s{{/strong}} looks pro and helps customers trust you. We've partnered with Google to offer you email, storage, docs, calendars, and more integrated with your site.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -85,7 +85,7 @@ const GSuiteUpsellCard = ( { | |||
<CompactCard> | |||
<header className="gsuite-upsell-card__header"> | |||
<h2 className="gsuite-upsell-card__title"> | |||
{ translate( 'Add Professional email from G Suite by Google Cloud to %(domain)s', { | |||
{ translate( 'Add professional email from G Suite by Google Cloud to %(domain)s', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🆗 This change will be queued for retranslation. We'll use the existing translations in the meantime.
438670b
to
417f03b
Compare
@danhauk: a heads-up that I've rebased this branch against
I've fixed that page.
@michelleweber: I've implemented that change (except for If both of you are ok with those changes, let's merge them 🙂. |
Looks great, thanks for taking that @stephanethomas! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works great, and the copy is much more relatable.
Looks great! |
Changes proposed in this Pull Request
This updates the copy on the G Suite stats nudge to remove "click here" in the heading. Changes
Customers can’t reach you at contact@%s - click here to add a mailbox
->Customers can’t reach you at contact@%s. Add a mailbox.
It looks like there was an A/B test in #31328 that settled on this copy. However, using "click here" for this heading is not great language.
This copy is only a suggestion. Pinging @Automattic/editorial and @Automattic/catalyst for input.
Testing instructions
After
screenshot above.