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

"Stop after this song" via CLI or FIFO (~/.quodlibet/control) #1909

Closed
Jimi-James opened this Issue Apr 6, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@Jimi-James

Jimi-James commented Apr 6, 2016

I need to assign the "Stop after this song" feature to a keyboard shortcut in my desktop. Right now, I have a script that uses xdotool to make the quodlibet window show up for a second, press the Shift+Space hotkey, and hide it again, but for reasons I cannot figure out through extensive testing, this doesn't work almost half the time, which defeats the entire purpose of the hotkey, because I have to use the right-click menu to check that it worked anyway. It'd be simpler if there was just a direct command line option to do it that I could put in that script.

@lazka lazka added the enhancement label Apr 6, 2016

@lazka lazka closed this in c3ad305 Apr 15, 2016

@Jimi-James

This comment has been minimized.

Jimi-James commented Apr 15, 2016

This is great! But, it's not quite enough. --stop-after=0|1 cannot be easily scripted, because it's not a toggle. The best I can do is make a file that tries to match the --stop-after status, i.e. if the file exists, stop-after is turned on, and if not, it's turned off, and do things like delete the file when quodlibet starts or quits or when the song changes. It's proving just as finnicky as the xdotool method. I still don't have a reliable script to toggle --stop-after until there's a toggleable option or a way to "get" the current status of stop-after.

@lazka lazka reopened this Apr 16, 2016

@lazka

This comment has been minimized.

Member

lazka commented Apr 16, 2016

What about --stop-after=0|1|t like --queue ?

@lazka lazka added the needinfo label Apr 16, 2016

@Jimi-James

This comment has been minimized.

Jimi-James commented Apr 16, 2016

That would work.

@lazka lazka removed the needinfo label Apr 17, 2016

@lazka lazka closed this in 2c4d32b Apr 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment