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

Issue with Unpausing withSpacebar. #2353

Open
levrx opened this issue May 18, 2024 · 3 comments
Open

Issue with Unpausing withSpacebar. #2353

levrx opened this issue May 18, 2024 · 3 comments
Labels
bug Something isn't working confirmed This bug was reproduced successfully by multiple people

Comments

@levrx
Copy link

levrx commented May 18, 2024

Description of the bug

You try to play, it pauses when you play.

Steps to reproduce

Go onto a show or movie, press the pause button with your mouse then try to unpause with your spacebar.

Expected behavior

You try to play, it pauses when you play.

Logs

No response

Screenshots

No response

Platform

Windows

Browser

Other (Please specify in the "additional context" section)

Jellyfin server version

10.9.2

Additional context

Brave
@levrx levrx added the bug Something isn't working label May 18, 2024
@manicExpressive
Copy link

manicExpressive commented Jun 21, 2024

Interesting bug! This seems to be a result of the pausePlayButton still being targeted in the DOM, so the space bar is doing the normal thing it does when nothing is targeted (pausing and playing the video) and also doing the click action on the pausePlayButton.

I'm not super deep on modern Vue so I went poking around and quickly realized doing a @click.once here on PlayPausebutton.vue wasn't the move as it would disable the play button after the first click!

A little special casing for the space bar might be the best way to tackle the bug, but this might also be a good spot for someone who loves keyboard shortcuts to add some finesse.

@ferferga
Copy link
Member

ferferga commented Jun 22, 2024

@manicExpressive @levrx Please give this a go at #2383

This is a quick solution I came up at this moment, though I think it's better if the global handlers simply focused and passed the event to the relevant buttons (like YouTube does). This is, however, a bit difficult at this moment due to Vuetify handling focus on it's own, instead of focusing the underlying button element. This is one of the reasons why I want to drop it and roll our own components based on it (faster updates and fix of these nnoying quirks).

@manicExpressive A new contributor would be awesome! 🎉

@manicExpressive
Copy link

Rad, thanks for taking a look - I followed up over on the PR with a quick suggestion, not super sure if their are broader implications beyond “works for me” :D

@ferferga ferferga added the confirmed This bug was reproduced successfully by multiple people label Oct 21, 2024
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 This bug was reproduced successfully by multiple people
Projects
None yet
Development

No branches or pull requests

3 participants