1302310 - Consider make video control's width match the size of video element

Status: VERIFIED FIXED

(Toolkit :: Video/Audio Controls, defect)

Description

[Ray Lin[:ralin]]

per comment 46 of bug 1271765, file another bug for the discussion.

Right now video control matches the size of video region inside video element, we should consider make video control match the size of video element.

Comment 1

[Carol Huang [:Carol]]

Attachment: preview.jpg

Hi Stephen,
Based on our offline discussion, could you comment your opinions about the video control set width? Currently the set width is following the video size. We would like to propose to change it to be fixed with the window length. (please see attachment)
Thanks!

Comment 2

[Stephen Horlander [:shorlander]]

Making the width of the video controls match the video container vs. the video region looks a lot nicer to me. It may also be slightly more technically correct since the controls apply to the entire container.

Comment 3

[Ray Lin[:ralin]]

Attachment: Bug 1302310 - Make the width of video control match video container.

Review commit: https://reviewboard.mozilla.org/r/94550/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/94550/

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8813025 [details]
Bug 1302310 - Make the width of video control match video container.

https://reviewboard.mozilla.org/r/94550/#review94994

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/23673ee58fea
Make the width of video control match video container. r=jaws

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/23673ee58fea

Comment 7

[Mihai Boldan, Desktop QA [:mboldan]]

I managed to reproduce this issue on Firefox 52.0, under Windows 10x64.
The issue is no longer reproducible on Firefox 53.0b2, under Ubuntu 16.04x64, Mac OS X 10.12.3, or under Windows 10x64.