842646 - Story - Context menu for video

Status: VERIFIED FIXED

(Tracking Graveyard :: Metro Operations, defect)

Comment 1

[Jim Mathies [:jimm]]

FWIW, I've removed the play and pause options from the video menu. With touch friendly controls in the overlay, I'm not sure why you need them.

Comment 2

[Asa Dotzler [:asa]]

Current implementation is: 

Save to video library
Copy video location
Open video in new tab

Yuan, what do you think about this. Is it sufficient? I wonder about full screen  and exit full screen. Ideally those would be in the main video controls toolbar, I think.

Comment 3

[Yuan Wang(:Yuan) – Mobile Firefox Design Lead]

Given the constraint on the number of menu items, the current implementation is sufficient. The order looks good to me. 

As long as the main video controls toolbar provides the basic options: play/pause,mute, full screen/exit full screen, the users should be offered all the basic controls for video.

Comment 4

[Asa Dotzler [:asa]]

(In reply to Yuan Wang(:Yuan) – Firefox UX Team from comment #3)
> Given the constraint on the number of menu items, the current implementation
> is sufficient. The order looks good to me. 
> 
> As long as the main video controls toolbar provides the basic options:
> play/pause,mute, full screen/exit full screen, the users should be offered
> all the basic controls for video.

Thanks. We're going to need to do something about the current video controls. They're the  fennec ones and they don't look nearly as good or complete as they do in desktop firefox. That's another story, though.

Comment 5

[Asa Dotzler [:asa]]

(In reply to Asa Dotzler [:asa] from comment #4)
> Thanks. We're going to need to do something about the current video
> controls. They're the  fennec ones and they don't look nearly as good or
> complete as they do in desktop firefox. That's another story, though.

Another story, and probably one for v2 I should have noted.

This story is reasonably well covered. Full-screen is being discussed in another bug and should not factor into verifying this.

To verify this story, ensure that the context menu on HTML5 video contains these working items in this order: 

Save to video library
Copy video location
Open video in new tab

Comment 6

[juan becerra [:juanb]]

Tested on 2013-03-05 using nightly built from http://hg.mozilla.org/mozilla-central/rev/015da7030aab
- Tested several sample html5 videos in MP4, WebM, and Ogg formats and made sure that the elements in the list and their order were as those described in comment #5.
- Tested these elements for functionality as well and those worked.

Comment 7

[Asa Dotzler [:asa]]

Attachment: Context menu for video

Comment 8

[Kamil Jozwiak [:kjozwiak]]

Re-verfied for it4 testing as per steps in Comment 6 and everything passes without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-04-02-03-08-43-mozilla-central/

Comment 9

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130808030205
Built from http://hg.mozilla.org/mozilla-central/rev/fd4cf30428b0

WFM
Tested on windows 8 using latest nightly for iteration-11. Followed steps provided in user story and got expected result.

Comment 10

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130823030204
Built from http://hg.mozilla.org/mozilla-central/rev/fb2318875cd4

WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in user story and got expected result.