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

Some keybinds don't work in mpv launched by invidtui #41

Open
paulie-g opened this issue Mar 9, 2024 · 3 comments
Open

Some keybinds don't work in mpv launched by invidtui #41

paulie-g opened this issue Mar 9, 2024 · 3 comments

Comments

@paulie-g
Copy link

paulie-g commented Mar 9, 2024

Config: latest Arch with invidtui from aur/invidtui-bin
Steps to reproduce: play any video with 'V', try to exit the player with 'q' (default mpv keybind for 'quit'), nothing happens

NB: thanks for creating this project

@darkhz
Copy link
Owner

darkhz commented Mar 9, 2024

Some default keybindings are blocked in an invidtui-launched mpv instance, like the quit keybinding, since all standard keybindings are usually handled within invidtui itself. To quit MPV, currently, you would use 'Shift+Q' within invidtui.

This was done so that while MPV exits, for example, the main invidtui instance would handle it and exit too.

This could be changed, however, like maybe, modifying MPV to reflect whatever defaults are set in invidtui?

@paulie-g
Copy link
Author

paulie-g commented Mar 9, 2024

Some default keybindings are blocked in an invidtui-launched mpv instance, like the quit keybinding, since all standard keybindings are usually handled within invidtui itself. To quit MPV, currently, you would use 'Shift+Q' within invidtui.

Yeah, I gave the code a quick look and it's clear. For the 'play audio' use case this is indeed the desired behaviour. However, it's not ergonomic for the 'play video' use case, especially if invidtui is going to crash when mpv terminates.

@darkhz
Copy link
Owner

darkhz commented Mar 10, 2024

Agreed. The 'play audio' case has been tested more than the 'play video' case, so it seems. I will look into and optimize the 'play video' case as well.

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

No branches or pull requests

2 participants