1346135 - Unable to play video on developer.android.com

Status: RESOLVED FIXED

(Core :: Layout, defect, P3)

Description

[Eric Tsai (no more review request accepted)]

From https://webcompat.com/issues/4993

Steps to reproduce:
1. https://developer.android.com/training/constraint-layout/index.html#add-a-constraint
2. Click on video

Expected result: Click on play button and play video

Actual result: Video doesn't play

https://developer.android.com/static/js/docs.js?v=2017031003 try add click/mouseover/mouseout event, these event doesn't work on Firefox video element but works on Chrome

Comment 1

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Hi Blake, as our quick discussion offline, could you please find someone to check if we implement those events for video elements? Thank you.

b.t.w. is it possible we get a minimum test case here?

Comment 2

[Blake Wu [:bwu][:blakewu]]

Alastor,
Can you have a look?

Comment 3

[Alastor Wu [:alwu]]

The video is the child of the div class "video-wrapper", I think the problem is onclick event doesn't deliver down to its child, the event was stuck on the div element.

If you remove the class "video-wrapper", the video can start playing well.

Comment 4

[Eric Tsai (no more review request accepted)]

Thanks :alwu, as @adamopenweb on https://github.com/webcompat/web-bugs/issues/4993#issuecomment-285757856, the pseudo element blocks pointer event:

>.video-wrapper.paused::before,
>.video-wrapper.playing::before {
>  bottom: 0;
>  content: "";
>  left: 0;
>  opacity: 0;
>  position: absolute;
>  right: 0;
>  top: 0;
>}

and @denschub's comment https://github.com/webcompat/web-bugs/issues/4993#issuecomment-285765833 is to add 
>pointer-events:none

to previous 2 classes. But the default behavior of pointer-event is different, we can see how to improve this.

Comment 5

[Blake Wu [:bwu][:blakewu]]

As denschub mentioned[1], this seems not a video bug, so change the component to be layout.

Comment 6

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

This is a P1 bug without an assignee. 

P1 are bugs which are being worked on for the current release cycle/iteration/sprint. 

If the bug is not assigned by Monday, 28 August, the bug's priority will be reset to '--'.

Comment 7

[Astley Chen (inactive)]

Looks like we are still waiting for a site fix to resolve the issue.

Comment 8

[Dennis Schubert [:denschub]]

The site has been fixed, so we can close this!