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

Consume arrow keys when changing playback rate #738

Open
angrave opened this issue Dec 4, 2023 · 2 comments
Open

Consume arrow keys when changing playback rate #738

angrave opened this issue Dec 4, 2023 · 2 comments
Assignees

Comments

@angrave
Copy link
Collaborator

angrave commented Dec 4, 2023

When changing playback rate using arrow keys, the video also jumps forwards and backwards. These events should be consumed by the control and not propagated to the player.

@mumbler6 mumbler6 self-assigned this Jun 26, 2024
@mumbler6
Copy link
Contributor

@angrave

Just to clarify, by the video jumping forward and backward, do you mean similar to how the left and right arrow keys skip 10 seconds back and forward? The playback rate shortcuts are on SHIFT+Up Arrow and Shift+Down Arrow, so changing the playback rate like this doesn't seem to jump the video forwards or backwards in that sense.

@angrave
Copy link
Collaborator Author

angrave commented Jul 2, 2024

Maybe this issue description is out of date. But there's still some quirky stuff.

  • Using shift versions I managed to get a displayed playback rate in the lower bar of "0.899999999" and also "x.0000...2". (I can't tell you how to reproduce these though, if I do, I'll update this issue... but it involves adding and subtracting a lot of deltas to exploit floating point error).
  • Another floating point error I managed to create: After adding and subtracting I managed to see a customized rate of "1.25" in the popup and the lower bar shows "1.25" but in the vertical options in the popup has "1" ticked, not 1.25 ticked
  • When the popup menu is displayed: Moving up to the customized control you can press left arrow (without the shift) to reduce the rate multiple times (as expected - you are controlling the UI element). However pressing right arrow increases the rate once but also shifts the focus to the vertical menu -so future attempts don't do anything. So you can only increment the rate once using the control, as the focus has now changed. I think right arrow event should be eaten at this point. i.e. use tab to leave the control.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants