Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bubbling Issues in IE8 #2414

Closed
mmcc opened this issue Jul 29, 2015 · 1 comment · Fixed by #2563
Closed

Bubbling Issues in IE8 #2414

mmcc opened this issue Jul 29, 2015 · 1 comment · Fixed by #2563

Comments

@mmcc
Copy link
Member

mmcc commented Jul 29, 2015

Noticed this while testing #2378, but event bubbling on the VolumeMenuButton menu seems to be busted in IE8 in master right now. Just tested to confirm, and it appears that the Captions menu is functioning as expected, so as far as I can tell, this seems to be VolumeMenuButton-specific.

To reproduce, try and use the volumeMenuButton in IE8 (in any configuration, inline or not). Things should appear to work normally until you finish adjusting the volume and mouseup. The volume will mute (or unmute) instead of staying at the intended location. Click, change volume, drag off the menu before mouseup, and the volume stays in the desired location.

@heff and I sprinkled a heavy dose of event console.logs around and so far the only thing we're sure of is that bubbling is broke in the component.

@heff
Copy link
Member

heff commented Aug 4, 2015

What we found is that no matter how much we did event.preventDefault/stopPropagation/bubbles:false, on all levels of the slider, the click still bubbled up to the main control component, which is the mute button. Need to keep digging to try to keep that from happening.

@heff heff added this to the v5.0.0 milestone Aug 24, 2015
heff added a commit to heff/video.js that referenced this issue Sep 7, 2015
fixes videojs#2414

Our fixEvent script makes a copy of the original event,
but it was failing to also set cancelBubble = true and
returnValue = false on the original event,
which is needed to make IE8 work like modern browsers.
@heff heff closed this as completed in #2563 Sep 7, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants