Code review comment for lp:~codygarver/elementaryos/fix-1248747

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

« Back to merge proposal