disable screensaver in fullscreen mode

Bug #1165885 reported by Denis Konstantinov
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Audience
Incomplete
Undecided
Unassigned

Bug Description

I implement disabling of screensaver in this branch
http://bazaar.launchpad.net/~linvinus/+junk/audience

also it contain:
fix: add audio tracks even if no information about codec or language name. (i forget mention this in commit)
fix decoding of file name, use glib functions instead manual parsing.
fix: don't use fixed width for tagview (otherwise buttons do not fit in allocated width, at least in Russian language)
new: hide tagview by mouse click (increase usability)

Revision history for this message
Denis Konstantinov (linvinus) wrote :

from gnome-settings-daemon 3.8 can be used freedesktop path https://bugzilla.gnome.org/show_bug.cgi?id=689225
(not present in "raring")

Revision history for this message
Tom Beckmann (tombeckmann) wrote :

Could you please move that branch over to audience and propose a merge?

In case you're not so familiar with launchpad, here's a quick howto:
- go to https://code.launchpad.net/~linvinus/+junk/audience
- On the right side, click "Change branch details"
- change "Branch Target" to project and the project to "audience"
- change the name to something more descriptive than audience, like screensaver-fix
- hit save
- go to code.launchpad.net/audience and click your branch, then click "Propose for merge"
- enter a description of your changes and submit

One thing that I saw by quickly looking over it, the indentation doesn't seem to be perfectly fine everywhere. That'd need to be fixed before we can merge it.

Revision history for this message
Denis Konstantinov (linvinus) wrote :

Thanks!
done.

Revision history for this message
Denis Konstantinov (linvinus) wrote :

oops, audience already have interaction with screensaver via X11.

sorry, my fault, i will move dbus implementation into VideoPlayer object.

Revision history for this message
Denis Konstantinov (linvinus) wrote :

done.
i resubmit proposal to merge.

Revision history for this message
Artem Anufrij (artem-anufrij) wrote :

I think it's already implemented in VideoPlayer.vala:

X.Display dpy; int timeout = -1; int interval; int prefer_blanking; int allow_exposures;
void set_screensaver (bool enable)

Changed in audience:
status: New → Incomplete
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.