Merge lp://staging/~vanvugt/ubuntu-system-settings/fix-1607240 into lp://staging/ubuntu-system-settings

Proposed by Daniel van Vugt
Status: Merged
Approved by: Jonas G. Drange
Approved revision: 1681
Merged at revision: 1735
Proposed branch: lp://staging/~vanvugt/ubuntu-system-settings/fix-1607240
Merge into: lp://staging/ubuntu-system-settings
Diff against target: 21 lines (+2/-2)
1 file modified
plugins/mouse/Connected.qml (+2/-2)
To merge this branch: bzr merge lp://staging/~vanvugt/ubuntu-system-settings/fix-1607240
Reviewer Review Type Date Requested Status
Jonas G. Drange (community) Approve
Lukáš Tinkl (community) Approve
Review via email: mp+301606@code.staging.launchpad.net

Commit message

Allow scroll speed multipliers up to 3.0; since the default speed in
gsettings-ubuntu-touch-schemas is changing from 0.5 to 1.0, some users
will want to ability to increase it above the native speed still.
(LP: #1607240) (LP: #1627567)

Just don't make the maximum too high or else the slider will be difficult
to use, since 1.0 is the native speed.

To post a comment you must log in.
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Maybe 3 would be better?...

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Yeah I also think 5.0 is a bit too much

1681. By Daniel van Vugt

Maximum scroll speed of 3.0 instead

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

LGTM, makes sense together with the other patches

review: Approve
Revision history for this message
Jonas G. Drange (jonas-drange) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches