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

minor annoyances #71

Open
wjtk4444 opened this issue Apr 27, 2022 · 2 comments
Open

minor annoyances #71

wjtk4444 opened this issue Apr 27, 2022 · 2 comments

Comments

@wjtk4444
Copy link

wjtk4444 commented Apr 27, 2022

Hey, I've been using playlistmanager for a while now and I wanted to share a list of small annoyances / mini-feature requests that I believe could be easily added (I hacked them in myself but I'm sure it could be handled in a more elegant way, hence why I'm opening and issue instead of a PR).

1. Add an option (or replace default behavior) to hide playlistmanager when calling playfile on currently played file.

Assuming default keybinds, I'd expect Enter -> Enter to toggle playlistmanager on and off without reloading currently played file.

Workaround/hack: check for cursor == pos in playfile() that removes keybinds and exits

  1. Re-work/fix toggle option.

I would expect toggle to toggle playlist view indefinitely instead of showing it for playlist_display_timeout seconds.

Workaround/hack: high timeout value (as mentioned in playlistmanager.conf)

  1. Allow for displaying playlist non-interactively (without adding keybinds).

I wrote some custom scripts that are supposed to display playlist in a purely informative, non-interactive way that does not overwrite input.conf keybinds.

Workaround/hack: yet another argument for script-message, ie.
playlistmanager show playlist [toggle|duration] [noninteractive].

  1. Better scaling with window size.

I don't know enough about ASS to say anything meaningful here, but I've checked how mpv's built-in OSC handles scaling and results are easier to read and with less visual artifacts (using same font and trying to match rendered font size).

  1. Allow for case-insensitive filename A-Z filename sorting (and make it default for alphanum sort?)
@FichteFoll
Copy link

Assuming default keybinds, I'd expect Enter -> Enter to toggle playlistmanager on and off without reloading currently played file.

The default binding to open (and close) the playlist is shift+enter and does what you propose.

Better scaling with window size.

Yeah, that would be helpful. Depending on the window aspect ratio, sometimes the playlist items can be unreadable.

@wjtk4444
Copy link
Author

wjtk4444 commented May 4, 2022

The default binding to open (and close) the playlist is shift+enter and does what you propose.

Nicely spotted, I just read it up and I could use it from within a script too. Thank you!

The name argument should be a short symbolic string. It allows the user to remap the key binding via input.conf using the script-message command, and the name of the key binding (...)

from mpv.io/manual

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