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

Videojs controls autohide #5258

Closed
mysuf opened this issue Jun 16, 2018 · 1 comment
Closed

Videojs controls autohide #5258

mysuf opened this issue Jun 16, 2018 · 1 comment
Labels

Comments

@mysuf
Copy link

@mysuf mysuf commented Jun 16, 2018

Hi guys,

I have one suggestion for possible improvement. It is logic behind controls bar showing/hiding. Now, it has really confusing behaviour that gradually increases user's rage and decreases user's experience. It disapears even if cursor is placed on control bar because of 2s hard timeout. So user is going to click, but it fades out meantime but shows again 1s later, because cursor probably trigger event on underlying player's DOM (this actually loops). I would expect behaviour like:

  1. cursor moves over player -> user active
  2. cursor stops inside player's bounds(outside control bar) -> sets timeout 2s before going inactive
  3. cursor stops inside control bar -> nothing happens or increased timeout about 10s
  4. cursor leaves player bounds -> sets timeout 2s before going inactive

Thanks

Version: 7.0.3
Player state: playing

@gkatsev
Copy link
Member

@gkatsev gkatsev commented Jun 22, 2018

Thanks for the issue. There's definitely room for improvement here and you've provided some interesting ideas. I'm currently working on a roadmap for Video.js and there's already some stuff related to this there, so, definitely going to reference this issue there.

@gkatsev gkatsev added the pinned label Jul 2, 2018
xjoaoalvesx added a commit to xjoaoalvesx/video.js that referenced this issue Dec 17, 2018
Handles the 'mouseenter' and 'mouseleave' events when triggered in the control-bar
Closes videojs#5258
@gkatsev gkatsev closed this in 6ebc772 Dec 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants