Merge lp://qastaging/~mterry/gsettings-ubuntu-touch-schemas/input-schema into lp://qastaging/gsettings-ubuntu-touch-schemas

Proposed by Michael Terry
Status: Merged
Approved by: Ken VanDine
Approved revision: 60
Merged at revision: 59
Proposed branch: lp://qastaging/~mterry/gsettings-ubuntu-touch-schemas/input-schema
Merge into: lp://qastaging/gsettings-ubuntu-touch-schemas
Diff against target: 96 lines (+73/-0)
3 files modified
accountsservice/Makefile.am (+1/-0)
accountsservice/com.ubuntu.AccountsService.Input.xml (+66/-0)
debian/changelog (+6/-0)
To merge this branch: bzr merge lp://qastaging/~mterry/gsettings-ubuntu-touch-schemas/input-schema
Reviewer Review Type Date Requested Status
Ken VanDine Approve
Michael Zanetti (community) Approve
Review via email: mp+283546@code.qastaging.launchpad.net

Commit message

Add com.ubuntu.AccountsService.Input schema.

Description of the change

This list of properties came from Ken (https://code.launchpad.net/~ken-vandine/unity8/mouse_touchpad_schema/+merge/281643).

= AccountsService =

But I've moved them to this more central location. And I've switched them to be AS properties.

I figured AS made more sense than gsettings because consider the encrypted-home case on the phone: a single user system, but now before the first login after booting, we change the user's mouse speed on them, and enable the touchpad while typing when they had it disabled. The user would be confused why we're losing settings.

Even in a multi-user case, some of these properties would want to be honored per-selected-user. Like maybe TouchpadDisableWhileTyping.

AS is a good place to keep these properties, so that the greeter can grab them if needed.

= Cursor Speed =

I've also updated the default for CursorSpeed to 0.75 from 0.5. It felt better to me (and we had complaints about 0.5 -- bug 1536383).

To post a comment you must log in.
Revision history for this message
Michael Zanetti (mzanetti) wrote :

ack!

review: Approve
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks good

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