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

Up Next container buttons not clickable #5650

Closed
pvico12 opened this issue Jun 3, 2024 · 4 comments
Closed

Up Next container buttons not clickable #5650

pvico12 opened this issue Jun 3, 2024 · 4 comments
Labels
bug Something isn't working confirmed The issue has been verified ui & ux This PR or issue mainly concerns UI & UX

Comments

@pvico12
Copy link

pvico12 commented Jun 3, 2024

Describe The Bug
Up Next Video container buttons not clickable. Instead they fast forward./backwards.

Steps To Reproduce

  1. Watch any media in a TV series until the last minute of the episode. Up next container appears.
  2. Click any button in the Up Next container.

Expected Behavior
Buttons should perform their regular operation.

System (please complete the following information):

  • Platform: Windows
  • Browser: Chrome
  • Jellyfin Version: 10.9.0
@pvico12 pvico12 added the bug Something isn't working label Jun 3, 2024
@thornbill
Copy link
Member

Are you using any custom css? #5595 (comment)

@pvico12
Copy link
Author

pvico12 commented Jun 3, 2024

@thornbill yes, adding a z-index to bring the dialog to the front solves the issue, but I did miss that issue so I opened a duplicate issue

@pvico12
Copy link
Author

pvico12 commented Jun 3, 2024

will look into hiding OSD instead of applying custom css

@dmitrylyzo
Copy link
Contributor

dmitrylyzo commented Jun 3, 2024

I found the problem:
Sometimes (maybe in some browsers) onHideAnimationComplete is called on btnPause, and the event listener is disconnecting because it was connected with once: true.
As a result, the hide class is not added to the OSD element, allowing the user to interact with transparent elements.

As for the use of z-index, we should avoid it if possible. Currently we can probably move upNextContainer after videoOsdBottom.

<div class="upNextContainer hide"></div>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed The issue has been verified ui & ux This PR or issue mainly concerns UI & UX
Projects
None yet
Development

No branches or pull requests

3 participants