1129108 - [FTE] Update Video for Vertical Scrolling

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Description

[Eric Pang [:epang] UX]

Hi Sam,

Can you help update the videos with the updated ones at this box link?

https://mozilla.box.com/s/2qmwji8c1lxtdo4bcd1w8uq81lbjb5pa

Needed to make a few visuals changes since the implementation of app grouping changed the home screen.

Please feel free to redirect and let me know if you have any questions.
Thanks!

Comment 1

[GitHub Autolander Bot]

Attachment: [PullReq] sfoster:ftu-tutorial-video-update-bug-1129108 to mozilla-b2g:master

Comment 2

[Sam Foster [:sfoster] (he/him)]

Attachment: Updated homescreen video assets

Comment 3

[Sam Foster [:sfoster] (he/him)]

Comment on attachment 8561565 [details] [review]
[PullReq] sfoster:ftu-tutorial-video-update-bug-1129108 to mozilla-b2g:master

Simple video asset update, r=me

Comment 4

[Sam Foster [:sfoster] (he/him)]

Merged to master: https://github.com/mozilla-b2g/gaia/commit/2bef75b40bc570400f7d9180bea615148aca49a6

Comment 5

[Sam Foster [:sfoster] (he/him)]

Comment on attachment 8561565 [details] [review]
[PullReq] sfoster:ftu-tutorial-video-update-bug-1129108 to mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): FTU Tutorial
[User impact] if declined: Vertical scroll video is inconsistent with the actual homescreen
[Testing completed]: Clicked through tutorial on device (Flame) and Gaia-Try run
[Risk to taking this patch] (and alternatives if risky): Low risk, just an asset swap
[String changes made]: None

Comment 6

[Ryan VanderMeulen [:RyanVM]]

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