Merge lp:~azzar1/caffeine/notify into lp:caffeine
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Reuben Thomas | ||||
Proposed branch: | lp:~azzar1/caffeine/notify | ||||
Merge into: | lp:caffeine | ||||
Diff against target: |
36 lines (+7/-7) 1 file modified
caffeine/core.py (+7/-7) |
||||
To merge this branch: | bzr merge lp:~azzar1/caffeine/notify | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Reuben Thomas | Pending | ||
Review via email: mp+49630@code.launchpad.net |
Description of the change
Many users complained that caffeine also disables notifications.
If we replace the freedesktop.org specific org.gnome.
We can also add the possibility to choice between the two option: with notification, without notification.
Unmerged revisions
- 366. By a
-
Many users complained that caffeine also disables notifications.
If we replace the freedesktop.org specific org.gnome.ScreenSaver
with the freedesktop.org specific org.gnome.SessionManager
we don't have this problem. I do not know if there is a special
reason for the choice of org.gnome.ScreenSaver but with this patch
Caffeine works the same.
This should be totally merged.