Custom controls not clickable in Fullscreen
Reported by
ggre...@gmail.com,
Jun 7 2018
|
||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.181 Safari/537.36 Steps to reproduce the problem: 1. Go to http://iandevlin.github.io/mdn/video-player/ 2. Click the mute button to mute the video 3. Click the Fullscreen button to enter fullscreen mode 4. Click the mute button again, the sound should be enabled again What is the expected behavior? When in fullscreen mode the custom controls should work the same as in non-fullscreen mode What went wrong? Custom controls are not working when fullscreen mode is enabled Did this work before? Yes Nov 2017 Does this work in other browsers? Yes Chrome version: 66.0.3359.181 Channel: n/a OS Version: 10.0 Flash Version: The custom controls on fullscreen mode work as expected in Mozilla Firefox
,
Jun 8 2018
Able to reproduce the issue on Windows 10, mac 10.13.3 and Ubuntu 17.10 using chrome reported version #66.0.3359.181 and latest canary #69.0.3452.0. Note: None of the controls are visible at fullscreen on latest canary #69.0.3452.0. Bisect Information: ===================== Good build: 64.0.3282.30 Bad Build : 64.0.3282.31 Change Log URL: (From Omahaproxy) https://chromium.googlesource.com/chromium/src/+log/64.0.3282.30..64.0.3282.31?pretty=fuller&n=10000 From the above change log suspecting below change Change-Id: Icde796405fca96e910480aef6f0d6be835f7a27a Reviewed-on: https://chromium-review.googlesource.com/788052 foolip@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Thanks...!!
,
Jun 8 2018
krajshree@ The controls appear to be fine for me in Canary on Linux and Windows.
,
Jun 8 2018
The video element being requested is the video element and the controls are outside of that element. If you adjust the code to request the videoContainer element fullscreen as fullscreen as opposed to the video element then it works correctly. This code is already followed for unprefixed fullscreen mode, mozilla and IE. The odd case is WebKit (not sure the comment about Safari 5.1 is still relevant but definitely Chrome has changed its implementation here). |
||||
►
Sign in to add a comment |
||||
Comment 1 by krajshree@chromium.org
, Jun 8 2018