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

Not starting when video is paused #8

Open
utack opened this issue Sep 14, 2016 · 7 comments
Open

Not starting when video is paused #8

utack opened this issue Sep 14, 2016 · 7 comments

Comments

@utack
Copy link

utack commented Sep 14, 2016

When a video is paused, the screensaver fails to start

https://paste.ubuntu.com/23178453/

Hope you know how to fix it, help is really welcome

@enen92
Copy link
Owner

enen92 commented Sep 14, 2016

Hum it will be a bit tricky since this screensaver also uses the player. So for it to be active it will need to stop the playback of your previous stream. This will carry some issues if you want to resume your video after stopping the screensaver (specially if using addons). But I'll have a look later.

@utack
Copy link
Author

utack commented Sep 14, 2016

Ok that is what I had feared
Hope it is easy to solve, would be nice to have the screensaver running when video is paused
Thanks for taking a look

@tremby
Copy link

tremby commented Jan 11, 2017

I thought other screensavers didn't start when a video is paused anyway. If I remember that correctly, it wouldn't be expected for this one to start in that situation.

@utack
Copy link
Author

utack commented Jan 11, 2017

The "dim" screensaver does
Maybe it is an exception?

@tremby
Copy link

tremby commented Jan 11, 2017

Oh, good point. Honestly I could be misremembering, but I'm pretty sure in the years I ran a slideshow screensaver it didn't start when a video was paused. Personally I wouldn't want it to, but it's possible that was a decision of that screensaver plugin, or an option for the user.

@scottluker
Copy link

I would personally love to see this impliented as well, but as an option for the user to select. Anyway love what you've done with this project

@enen92
Copy link
Owner

enen92 commented Feb 6, 2017

there's a reason for this issue not be closed already :) I plan to do it...as an optional setting since I find it "intrusive" myself

@enen92 enen92 closed this as completed Feb 6, 2017
@enen92 enen92 reopened this Feb 6, 2017
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

4 participants