Merge lp:~codygarver/elementaryos/fix-1248747 into lp:~elementary-os/elementaryos/default-settings-trusty

Proposed by Cody Garver
Status: Merged
Merged at revision: 103
Proposed branch: lp:~codygarver/elementaryos/fix-1248747
Merge into: lp:~elementary-os/elementaryos/default-settings-trusty
Diff against target: 13 lines (+4/-0)
1 file modified
debian/elementary-default-settings.gsettings-override (+4/-0)
To merge this branch: bzr merge lp:~codygarver/elementaryos/fix-1248747
Reviewer Review Type Date Requested Status
Sergey "Shnatsel" Davidoff (community) Abstain
Review via email: mp+194260@code.launchpad.net

Commit message

Set org.gnome.settings-daemon.plugins.cursor active false to fix bug #1248747

Description of the change

I don't know what effects this could have

To post a comment you must log in.
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

According to upstream bug (https://bugzilla.gnome.org/show_bug.cgi?id=694758) this is caused by a bug in touchscreen detection. Presumably caused by this commit: https://git.gnome.org/browse/gnome-settings-daemon/commit/?id=285b0e497af18b853c9f67f93f3b3bbab7a6c241

I have no idea what other things applying this workaround is going to break. If you feel it's really necessary, please don't mark the bug fixed until it's fixed upstream and a proper fix is applied.

Also, if you're going to merge this, please document why and link to both LP and upstream bug reports in the comment next to the newly introduced config.

review: Abstain
104. By Cody Garver

Document workaround

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/elementary-default-settings.gsettings-override'
2--- debian/elementary-default-settings.gsettings-override 2013-10-26 02:17:37 +0000
3+++ debian/elementary-default-settings.gsettings-override 2013-11-07 12:32:21 +0000
4@@ -108,6 +108,10 @@
5 natural-scroll=true
6 scroll-method='two-finger-scrolling'
7
8+[org.gnome.settings-daemon.plugins.cursor]
9+# Workaround upstream gnome-settings-daemon bug (https://bugzilla.gnome.org/show_bug.cgi?id=694758) as tracked by elementary (https://bugs.launchpad.net/elementaryos/+bug/1248747)
10+active=false
11+
12 [org.gnome.settings-daemon.plugins.media-keys]
13 screensaver='<Super>l'
14

Subscribers

People subscribed via source and target branches