-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Controls not fading correctly on mobile #2886
Comments
For a mouse, the behavior should be:
For touch screens, the behavior should be:
However, there seems to be a small bug on mobile. In my manual testing, I found:
Does this help explain how things should work? I will label this "bug" for now and try to fix the issue I just mentioned. Would this resolve your problem? |
Thanks for the explanation for all the scenarios. But does the bug you reported will fix the case when we click on any other item in the controls like seek bar or volume or any other custom controls (other than overflow menu). As in mobile view if we click on the above-mentioned controls, then they do not fade away until clicked somewhere else. |
I can reproduce that, yes. Something odd is happening, and I don't understand it yet. For some reason, on mobile, using touch events, the controls panel winds up in a "hover" state according to the browser. I found a fix for this in the overflow menus, but it is still happening for other controls. I didn't notice the effect on, say, mute when I initially tested it. |
Found another solution that works for everything. Once I get the changed through code review, they will be published to GitHub. Thanks! |
Conflicting touch event listeners in OverflowMenu and Controls led to a weird situation in which the controls container was seen as being hovered after tapping to close the overflow menu. This, in turn, led the controls to stay open instead of hiding. Removing the extra event listener in OverflowMenu allows the controls to be properly hidden when the overflow menu is closed. Issue #2886 Change-Id: I18688efbcc6c4129b7b03e9daf00964d3a8d9802
Conflicting touch event listeners in OverflowMenu and Controls led to a weird situation in which the controls container was seen as being hovered after tapping to close the overflow menu. This, in turn, led the controls to stay open instead of hiding. Removing the extra event listener in OverflowMenu allows the controls to be properly hidden when the overflow menu is closed. Issue #2886 Change-Id: I18688efbcc6c4129b7b03e9daf00964d3a8d9802
On pointerless touch-screen devices, :hover queries on an element do not make sense. In spite of this, they can pass on such devices after a touch event, and elements even get stuck in this false :hover state. Now we check if the device supports hovering before making the :hover query on individual elements, which fixes the hiding on controls on mobile after a touch event. Closes #2886 Change-Id: I5ae0b0c84c69961c33df5e50f480b53495ab2694
Conflicting touch event listeners in OverflowMenu and Controls led to a weird situation in which the controls container was seen as being hovered after tapping to close the overflow menu. This, in turn, led the controls to stay open instead of hiding. Removing the extra event listener in OverflowMenu allows the controls to be properly hidden when the overflow menu is closed. Issue #2886 Backported to v2.5.x Change-Id: I18688efbcc6c4129b7b03e9daf00964d3a8d9802
On pointerless touch-screen devices, :hover queries on an element do not make sense. In spite of this, they can pass on such devices after a touch event, and elements even get stuck in this false :hover state. Now we check if the device supports hovering before making the :hover query on individual elements, which fixes the hiding on controls on mobile after a touch event. Closes #2886 Backported to v2.5.x Change-Id: I5ae0b0c84c69961c33df5e50f480b53495ab2694
Thanks for the help. Can you reply once its pushed? |
The fix has already been pushed to the v3.0.x branch, and we are working to push out v3.0.5 very soon. |
Just a request we are using 2.5.10 majorly on our application. Is it possible to give a patch release for that version? |
The fix has been cherry-picked for both v2.5.17 and v3.0.5. |
Both releases are live on GitHub and NPM already. There is an issue with our deployment to Google Hosted Libraries, which has delayed the release announcement email. |
Have you read the Tutorials?
yes
Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
3.0.4 and 2.5.10
Please ask your question
Is there a way to fade away the controls after some time in shaka player?
Currently, if we hover on seek bar or any other controls on the bottom menu the controls does not fade away after some time.
It only hides if we move the mouse somewhere out. While in mobile view it only fades when we touch somewhere outside.
Can you suggest a way to do so?
The text was updated successfully, but these errors were encountered: