Closed Bug 1130633 Opened 9 years ago Closed 3 years ago

Video controls slow to respond and don't actually control video that auto played

Categories

(Firefox for Android Graveyard :: General, defect)

38 Branch
All
Android
defect
Not set
normal

Tracking

(fennec+)

RESOLVED INCOMPLETE
Tracking Status
fennec + ---

People

(Reporter: krudnitski, Unassigned)

References

()

Details

Attachments

(1 file)

On the telegraph.co.uk site which gives me mobile version (great). 

Loaded up an article and started to read it. Suddenly, I hear a commentator! I swear a little because I am in bed trying not to wake my sleeping toddler or teething baby up.

I scroll down, find the culprit, and tap on pause. Controls are slow to respond so I tap repeatedly and harder. Finally it works. But the video keeps playing!! I have to exit the article to make it stop.
tracking-fennec: --- → ?
This seems to only be a problem with this specific Ooyala player on this site.
Mike, sounds like a web-compat issue. What to have a look?
Flags: needinfo?(miket)
Yeah, I can see the issue. It's kind of finnicky, sometimes the click on play/pause works, sometimes it needs 3 or 4 taps to do what I want. Looking closer...
Digging into the ooyala v3 code at http://player.ooyala.com/v3/ZTIxYmJjZDM2NWYzZDViZGRiOWJjYzc5?platform=html5-fallback, I don't see anything that looks super alarming.

It *seems* like what is happening is when this bug happens, the touch/click events don't capture inside <div class="oo_controls_inner vod"></div>, which contains the play/pause buttons. It's strange... I'm going to see if I can reduce this to a testcase because it's a PITA to load 50 ads for every little tweak/test I try.
Based on https://miketaylr.com/bzla/ooyala.html, I think this is closer to our bug than anything Ooyala or the site is doing. 

#playerwrapper .oo_mini_controls .oo_toolbar_item {
    /*default size:*/
    /*width: 16px;
    height: 16px;*/

    width: 30px !important;
    height: 30px !important;
}

When the pause/play buttons are 16px wide/high, we just don't hit them very much. If you bump that to 30px or larger, it works as expected and just like Chrome.

Aaron, can you move this to the right component?
Component: Audio/Video → General
Flags: needinfo?(aaron.train)
Sounds like we could work better if we tuned our "fuzzy hit testing" code to find something about the play/pause buttons to trigger our "send the tap to that element" code path.
Mentor: wjohnston
tracking-fennec: ? → +
Mentor: wjohnston2000
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: