We'll have updated tours on version 57 which needs different welcome messages. UX spec string followup needed for: - Updated user tip for overlay icon - Updated user tip for notification

See bug 1360378 comment 5 for the follow-up string. The string is current in review now so I'll just come up a WIP until they are locked down.

So here comes the WIP.

The strings has been locked down per bug 1360378 comment 5 so let's step forward. I'll give out a screenshot later.

Comment on attachment 8886948 [details] Bug 1380963 - Display different welcome messages for update or new users on the overlay tour icon. https://reviewboard.mozilla.org/r/157714/#review163360 ::: browser/extensions/onboarding/locales/en-US/onboarding.properties:13 (Diff revision 2) > onboarding.button.learnMore=Learn More > # LOCALIZATION NOTE(onboarding.overlay-icon-tool-tip): This string will be used to show the tooltip alongside the notification icon. %S is brandShortName. > onboarding.overlay-icon-tool-tip=New to %S? Let’s get started. > # LOCALIZATION NOTE(onboarding.notification-icon-tool-tip): This string will be used to show the tooltip alongside the notification icon. %S is brandShortName. > onboarding.notification-icon-tool-tip=New to %S? > +# LOCALIZATION NOTE(onboarding.overlay-icon-tool-tip-updated): %S is brandShortName. nit: tooltip is one word, so I'd avoid tool-tip in these string IDs ::: browser/extensions/onboarding/locales/en-US/onboarding.properties:15 (Diff revision 2) > onboarding.overlay-icon-tool-tip=New to %S? Let’s get started. > # LOCALIZATION NOTE(onboarding.notification-icon-tool-tip): This string will be used to show the tooltip alongside the notification icon. %S is brandShortName. > onboarding.notification-icon-tool-tip=New to %S? > +# LOCALIZATION NOTE(onboarding.overlay-icon-tool-tip-updated): %S is brandShortName. > +onboarding.overlay-icon-tool-tip-updated=%S is all new. See what you can do! > +onboarding.notification-icon-tool-tip-updated=See what's new! Use proper apostrophe ’, otherwise this will fail tests.

(In reply to Francesco Lodolo [:flod] from comment #8) > nit: tooltip is one word, so I'd avoid tool-tip in these string IDs To clarify: I'd use icon-tooltip instead of icon-tool-tip

Comment on attachment 8886948 [details] Bug 1380963 - Display different welcome messages for update or new users on the overlay tour icon. https://reviewboard.mozilla.org/r/157714/#review163416 ::: browser/extensions/onboarding/content/onboarding.js:656 (Diff revision 2) > <button id="onboarding-notification-action-btn"></button> > </section> > <button id="onboarding-notification-close-btn"></button> > `; > - let toolTip = this._bundle.formatStringFromName("onboarding.notification-icon-tool-tip", [BRAND_SHORT_NAME], 1); > + let toolTip = this._bundle.formatStringFromName( > + this._tourType == "new" ? "onboarding.notification-icon-tool-tip" === ::: browser/extensions/onboarding/content/onboarding.js:706 (Diff revision 2) > > _renderOverlayIcon() { > let icon = this._window.document.createElement("button"); > icon.id = "onboarding-overlay-icon"; > - let tooltip = this._bundle.formatStringFromName("onboarding.overlay-icon-tool-tip", [BRAND_SHORT_NAME], 1); > + let tooltip = this._bundle.formatStringFromName( > + this._tourType == "new" ? "onboarding.overlay-icon-tool-tip" === ::: browser/extensions/onboarding/content/onboarding.js:707 (Diff revision 2) > _renderOverlayIcon() { > let icon = this._window.document.createElement("button"); > icon.id = "onboarding-overlay-icon"; > - let tooltip = this._bundle.formatStringFromName("onboarding.overlay-icon-tool-tip", [BRAND_SHORT_NAME], 1); > + let tooltip = this._bundle.formatStringFromName( > + this._tourType == "new" ? "onboarding.overlay-icon-tool-tip" > + : "onboarding.overlay-icon-tool-tip-updated", lint: `:` should append at the previous line

Comment on attachment 8886948 [details] Bug 1380963 - Display different welcome messages for update or new users on the overlay tour icon. https://reviewboard.mozilla.org/r/157714/#review163992

Comment on attachment 8886948 [details] Bug 1380963 - Display different welcome messages for update or new users on the overlay tour icon. https://reviewboard.mozilla.org/r/157714/#review164362

Comment on attachment 8886948 [details] Bug 1380963 - Display different welcome messages for update or new users on the overlay tour icon. https://reviewboard.mozilla.org/r/157714/#review164418

hg error in cmd: hg push -r . -f try: pushing to ssh://hg.mozilla.org/try searching for changes remote: adding changesets remote: adding manifests remote: adding file changes remote: added 1 changesets with 0 changes to 0 files (+1 heads) remote: remote: remote: ************************** ERROR **************************** remote: Error accessing https://api.pub.build.mozilla.org/treestatus/trees/try : remote: HTTP Error 500: INTERNAL SERVER ERROR remote: Unable to check if the tree is open - treating as if CLOSED. remote: To push regardless, include "CLOSED TREE" in your push comment. remote: ************************************************************* remote: remote: remote: transaction abort! remote: rollback completed remote: pretxnchangegroup.a_treeclosure hook failed abort: push failed on remote

FWIW, our traditional way of using Target Milestone is that it corresponds to when code hits mozilla-central.

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/05e27c93b3b4 Display different welcome messages for update or new users on the overlay tour icon.r=flod,gasolin,mossop

(In reply to Ryan VanderMeulen [:RyanVM] from comment #19) > FWIW, our traditional way of using Target Milestone is that it corresponds > to when code hits mozilla-central. Got you. This feature is planned to be opened in version 57; We just planned our development schedule earlier. Otherwise there's a very short period before feature freeze. If that should correspond to the actual time patch landed, I can change it to 56.

FYI, this moved the welcome fox slightly to the right. See screenshots: https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=7d2e89fb92331d7e4296391213c1e63db628e046&newProject=mozilla-central&newRev=0faada5c2f308f101ab7c54a87b3dce80b97d0e3&filter=primaryUI Which was probably intended. :)

I updated from 55 to 57 and got the "New to Nightly" message. Is there a specific way to provoke these actions?

That's a little bit strange because you should get nothing for a 55-to-57 upgrade for now. (whereas after bug 1366056 landed later, it should shows correct set of updated tours). But in any way you need to trigger it manually for now. To trigger an updated tour manually, change the following two preferences: browser.onboarding.tour-type: set to "true". browser.onboarding.updatetour: set to some reasonable toursets like "performance,library,singlesearch,customize,sync".

Sorry for the typo. browser.onboarding.tour-type: set to "update". browser.onboarding.updatetour: set to some reasonable toursets like "performance,library,singlesearch,customize,sync".

Thanks Rex. I was able to verify this bug on today's nightly with Rex's steps in Comment 27.