ubuntu-touch-coreapps-reviewers team mailing list archive

[Niklas Wenzel <nikwen.developer@xxxxxxxxx>]

Niklas Wenzel has proposed merging lp:~nikwen/ubuntu-terminal-app/layouts-click-full-width into lp:ubuntu-terminal-app.

Commit message:
Allow triggering the layout enabled switch by clicking anywhere on the ListItem

Requested reviews:
  Ubuntu Terminal Developers (ubuntu-terminal-dev)

For more details, see:
https://code.launchpad.net/~nikwen/ubuntu-terminal-app/layouts-click-full-width/+merge/266667

Allow triggering the layout enabled switch by clicking anywhere on the ListItem
-- 
Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~nikwen/ubuntu-terminal-app/layouts-click-full-width into lp:ubuntu-terminal-app.

=== modified file 'src/app/qml/LayoutsPage.qml'
--- src/app/qml/LayoutsPage.qml	2015-03-26 22:32:38 +0000
+++ src/app/qml/LayoutsPage.qml	2015-08-02 13:14:30 +0000
@@ -35,13 +35,17 @@
         anchors.fill: parent
         model: settings.profilesList
         delegate: ListItem.Standard {
+            text: name
+
             control: Switch {
+                id: layoutSwitch
                 checked: profileVisible
                 onCheckedChanged: {
                     settings.profilesList.setProperty(index, "profileVisible", checked);
                 }
             }
-            text: name
+
+            onTriggered: layoutSwitch.trigger()
         }
     }
 }