Merge lp:~michael-sheldon/ubuntu-system-settings/custom-keyboard-opacity into lp:ubuntu-system-settings

Proposed by Michael Sheldon
Status: Merged
Approved by: Jonas G. Drange
Approved revision: 1749
Merged at revision: 1751
Proposed branch: lp:~michael-sheldon/ubuntu-system-settings/custom-keyboard-opacity
Merge into: lp:ubuntu-system-settings
Diff against target: 41 lines (+24/-0)
1 file modified
plugins/language/PageComponent.qml (+24/-0)
To merge this branch: bzr merge lp:~michael-sheldon/ubuntu-system-settings/custom-keyboard-opacity
Reviewer Review Type Date Requested Status
Jonas G. Drange (community) Approve
system-apps-ci-bot continuous-integration Approve
Review via email: mp+312959@code.launchpad.net

Commit message

Add keyboard opacity setting to language panel

Description of the change

Add keyboard opacity setting to language panel

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:1749
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/272/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/2172
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/560
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2175
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/2001/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/2001
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/2001/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/272/rebuild

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

Looks good, however, when I do $ gsettings list-keys com.canonical.keyboard.maliit---opacity does not show up. Is this key being added elsewhere?

review: Needs Information
Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

Yep, that's added in this MR: https://code.launchpad.net/~michael-sheldon/ubuntu-keyboard/custom-opacity/+merge/312960

Currently both are built together in silo 2289 for testing: https://bileto.ubuntu.com/#/ticket/2289

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

Very good, thank you Michael.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/language/PageComponent.qml'
2--- plugins/language/PageComponent.qml 2016-11-29 11:12:13 +0000
3+++ plugins/language/PageComponent.qml 2016-12-10 01:17:52 +0000
4@@ -27,6 +27,7 @@
5 import Ubuntu.Components.ListItems 1.3 as ListItem
6 import Ubuntu.Settings.Menus 0.1 as Menus
7 import Ubuntu.SystemSettings.LanguagePlugin 1.0
8+import Ubuntu.Settings.Components 0.1 as USC
9
10 ItemPage {
11 id: root
12@@ -281,6 +282,29 @@
13 onTriggered: settings.keyPressHapticFeedback = checked
14 }
15 }
16+
17+ Menus.SliderMenu {
18+ text: i18n.tr("Keyboard opacity")
19+
20+ id: opacity
21+ objectName: "opacity"
22+ function formatValue(v) { return v * 100 }
23+ minimumValue: 0.5
24+ maximumValue: 1
25+ value: settings.opacity
26+ live: true
27+
28+ property real serverValue: settings.opacity
29+ USC.ServerPropertySynchroniser {
30+ userTarget: opacity
31+ userProperty: "value"
32+ serverTarget: opacity
33+ serverProperty: "serverValue"
34+ maximumWaitBufferInterval: 16
35+
36+ onSyncTriggered: settings.opacity = value
37+ }
38+ }
39 }
40 }
41 }

Subscribers

People subscribed via source and target branches