Sam Foster [:sfoster] (he/him) Assignee
	Description • 10 years ago

The tutorial videos need to be updated before release (all resolution/px densities) to represent the UI the user will see. In particular the App Grouping feature and tutorial step is new to 2.2.

This should be an asset-only change, with no code or strings affected.
Videos live at:  gaia/apps/ftu/style/images/tutorial/*.mp4

	Eric Pang [:epang] UX
	Comment 1 • 10 years ago

Added fte videos for App Grouping here on box:
https://mozilla.box.com/s/8tf3vmhz37pcorp9mlf7n5qvjkz0td6c

	Ben Francis [:benfrancis]
	Comment 2 • 10 years ago

Triage: Sam, are you able to take this one?

	Sam Foster [:sfoster] (he/him) Assignee
	Comment 3 • 10 years ago

Yeah, I should've assigned myself, I'll be taking this

	Sam Foster [:sfoster] (he/him) Assignee
	Comment 4 • 10 years ago

Updates the App Grouping placeholder with the final assets. Remember you'll need the GAIA_DPPX=1.5 when you build/flash (as appropriate for your device) to see the higher quality video. 

Just needs a sanity check really. I've flashed to my Flame with a couple different GAIA_DPPX=1.5 values and all was as expected. 

Eric: Let me know if I can help you test this.

	Eric Pang [:epang] UX
	Comment 5 • 10 years ago

Comment on attachment 8553391 [details]
PR: Replace placeholder with final App Grouping video assets

Looks good to me, thanks for updating Sam!

	Fernando Campo (:fcampo)
	Comment 6 • 10 years ago

Comment on attachment 8553391 [details]
PR: Replace placeholder with final App Grouping video assets

yup, all good

	Sam Foster [:sfoster] (he/him) Assignee
	Comment 7 • 10 years ago

Merged to master: https://github.com/mozilla-b2g/gaia/pull/27607

	Sam Foster [:sfoster] (he/him) Assignee
	Comment 8 • 10 years ago

Comment on attachment 8553391 [details]
PR: Replace placeholder with final App Grouping video assets

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): FTU Tutorial / App Grouping feature
[User impact] if declined: Placeholder video, says "placeholder" 
[Testing completed]: Gaia-Try, manual testing on Flame with various GAIA_DPPX settings
[Risk to taking this patch] (and alternatives if risky): V. low, no code changes just the video assets update
[String changes made]: None

Dependent on bug 1122703

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 10 years ago

v2.2: https://github.com/mozilla-b2g/gaia/commit/91495dcfb373026cf6de29d859e5e2153c5bf10f

	Hermes Cheng (inactive after July 27, 2015)
	Comment 10 • 9 years ago

verified since already landed