Verdi [:verdi] Best to slack me Assignee
	Description • 7 years ago

When a user updates from <Firefox 57 to ≥57 we want to provide an update tour and notifications. We'll use the same design and interactions as the new user tour (Bug 1354707 and Bug 1357668) with the caveats that:
* The content will be different
* There may be a different amount of items in the tour
* Additional features of the UITour API may be required (e.g. highight a new Photon-only menu item) 

The specific content is TBD pending more information about the major features and function of Photon along with any potential user-facing Quantum items that need exposure.

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 1 • 7 years ago

(bug 1354707 comment 11 also applies here)

	Tim Guan-tin Chien [:timdream] (please needinfo)
	Comment 2 • 7 years ago

See bug 1354707 comment 12.

	KM Lee [:rexboy] (inactivate after Sep. 22)
	Comment 3 • 7 years ago

Just opened the corresponding bugs with Fred so clearing ni?.

	Francis Lee [:frlee]
	Comment 4 • 7 years ago

hi Michael,

could you please update the latest spec here? dev team has already started to implement 57 related features.

thank you very much

	Verdi [:verdi] Best to slack me Assignee
	Comment 5 • 7 years ago

The 57 Update Tour Overlay screens can be seen here (along with notification screens) https://mozilla.invisionapp.com/share/NHCLUXCM7#/243977044_01-Newtab

The illustrations for Performance, Library and Screenshots are just placeholders and will need to be replaced.

The navigation icon for performance is just a placeholder and will need to be replaced.

The copy for the new items is still being reviewed but you can view the document here - https://docs.google.com/a/mozilla.com/document/d/1abXULIwaMLnXawkLUjpcVawju8tmH0xlVGCOBhuVztM/edit?usp=sharing

	Verdi [:verdi] Best to slack me Assignee
	Comment 6 • 7 years ago

Nav icons for the added tours

	Verdi [:verdi] Best to slack me Assignee
	Comment 7 • 7 years ago

(In reply to Verdi [:verdi] from comment #5)
> The copy for the new items is still being reviewed but you can view the
> document here -
> https://docs.google.com/a/mozilla.com/document/d/
> 1abXULIwaMLnXawkLUjpcVawju8tmH0xlVGCOBhuVztM/edit?usp=sharing

This copy is approved.

	gasolin@mozilla.com
	Comment 8 • 7 years ago

I found the icon are 92x92 which other existing icons are 16x16, 

Does the icon is final, or can we use placeholder icon now and then update once the final icon is available?

	gasolin@mozilla.com
	Comment 9 • 7 years ago

another question is should we update the sync button to new string and use it for both v56/v57?

	KM Lee [:rexboy] (inactivate after Sep. 22)
	Comment 10 • 7 years ago

Excuse me for the third question :p
Will we have icons of new tours for notifications?

	KM Lee [:rexboy] (inactivate after Sep. 22)
	Comment 11 • 7 years ago

(In reply to Fred Lin [:gasolin] from comment #8)
> I found the icon are 92x92 which other existing icons are 16x16, 
> 
> Does the icon is final, or can we use placeholder icon now and then update
> once the final icon is available?

This icon has been put in the patch of fast&modern tour and is just about to be landed. I adjusted the size to 16x16.
If visual update is still required, please go to bug 1382520.

	Verdi [:verdi] Best to slack me Assignee
	Comment 12 • 7 years ago

(In reply to Fred Lin [:gasolin] from comment #8)
> I found the icon are 92x92 which other existing icons are 16x16, 
> 
> Does the icon is final, or can we use placeholder icon now and then update
> once the final icon is available?

I didn't think the size made a difference because they are SVGs. The spec has all of the navigation icons at 20x20 pixels. But yes, my intention was that the navigation icon for performance is just a placeholder and will need to be replaced.

(In reply to KM Lee [:rexboy] from comment #10)
> Will we have icons of new tours for notifications?
Yes, I commented here - bug 1382374 comment0 and bug 1382376 comment 0

(In reply to Fred Lin [:gasolin] from comment #9)
> another question is should we update the sync button to new string and use
> it for both v56/v57?

Confirming what we agreed in the meeting yesterday which was to use the 56 copy of "Next" on the sync button.

	Verdi [:verdi] Best to slack me Assignee
	Comment 13 • 7 years ago

I'm still waiting for new navigation icons for:
*Performance

	Verdi [:verdi] Best to slack me Assignee
	Comment 14 • 7 years ago

Nav icons posted in bug 1382520 comment4

	Justin [:JW_SoftvisionQA]
	Comment 15 • 7 years ago

Marking as verified since the specs were provided.