Merge lp:~unity-team/unity/phablet.dashBar_bottomswipe into lp:unity/phablet

[Andrea Cimitan]

Bottom swipe reveals the dash bar

Bottom swipe reveals the dash bar, integrating some logic from GenericToolbar.qml internal qml component used in the SDK

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:438
http://jenkins.qa.ubuntu.com/job/phablet-unity-ci/690/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/phablet-unity-ci/./build=panda-pbuilder,distribution=quantal,flavor=armhf/690/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/phablet-unity-ci/./build=pbuilder-master,distribution=quantal,flavor=i386/690/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/phablet-unity-ci/690//rebuild/?

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

misses tests

review: Needs Fixing

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

439. By Andrea Cimitan on 2013-03-18

Merged trunk

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:439
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/54/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/55/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/55/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/54/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/54/rebuild

review: Approve (continuous-integration)

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

440. By Andrea Cimitan on 2013-03-19

Merged trunk, modified unittests CMakeList, added WIP qmluitest DashBar

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:440
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/87/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/88/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/88/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/87/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/100/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/87/rebuild

review: Needs Fixing (continuous-integration)

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

441. By Andrea Cimitan on 2013-03-19

Some more tests

442. By Andrea Cimitan on 2013-03-19

Merged trunk

443. By Andrea Cimitan on 2013-03-19

Add slow swipe

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:443
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/91/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/92/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/92/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/91/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/115/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/91/rebuild

review: Approve (continuous-integration)

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

444. By Andrea Cimitan on 2013-03-19

Add test for hide after navigation

445. By Andrea Cimitan on 2013-03-20

Add test for itemSelected, added readonly property rowWidth for the wdth of the row inside the DashBar, made some properties readonly

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:445
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/99/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/100/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/100/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/99/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/149/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/99/rebuild

review: Approve (continuous-integration)

[Michał Sawicz (saviq)]

Can we please have a separate MR with just the CMake tweaks?

Also, this looks a lot like a copy of what's there for the Launcher, could we not think of a way of abstracting that behaviour?

review: Needs Fixing

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

446. By Michael Zanetti on 2013-03-20

make it even more weird

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:446
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/100/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/101/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/101/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/100/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/158/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/100/rebuild

review: Approve (continuous-integration)

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

447. By Andrea Cimitan on 2013-03-20

This is weird

448. By Andrea Cimitan on 2013-03-20

better debug

449. By Andrea Cimitan on 2013-03-20

Finished testing DashBar

450. By Andrea Cimitan on 2013-03-20

Revert old CMake for unittests

[Andrea Cimitan (cimi)]

> Can we please have a separate MR with just the CMake tweaks?
>
I extracted and pushed lp:~unity-team/unity/phablet.unittests_new-cmake

> Also, this looks a lot like a copy of what's there for the Launcher, could we
> not think of a way of abstracting that behaviour?

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:447
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/101/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/102/console
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/102/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/101/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/162/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/101/rebuild

review: Approve (continuous-integration)

[lp:~unity-team/unity/phablet.dashBar_bottomswipe]

451. By Andrea Cimitan on 2013-03-20

id: root is not used

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:450
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/104/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/105/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/104
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/164

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/104/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:451
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/106/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/107
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/107/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/106
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/169

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/106/rebuild

review: Approve (continuous-integration)

[Andrea Cimitan (cimi)]

> misses tests

not anymore! :-)

[Michael Zanetti (mzanetti)]

> > misses tests
>
> not anymore! :-)

Great! Testing looks ok. I couldn't verify because there are a bunch of merge conflicts by now. Anyways, once they are resolved its fine with me.

I think there is one Needs Fixing from Saviq left tho. The one about duplicating the dragging-code from the launcher.

[Michael Zanetti (mzanetti)]

Can we please either fix or delete this?

[Andrea Cimitan (cimi)]

There's a new branch using Panel SDK component

Unmerged revisions

451. By Andrea Cimitan on 2013-03-20

id: root is not used

450. By Andrea Cimitan on 2013-03-20

Revert old CMake for unittests

449. By Andrea Cimitan on 2013-03-20

Finished testing DashBar

448. By Andrea Cimitan on 2013-03-20

better debug

447. By Andrea Cimitan on 2013-03-20

This is weird

446. By Michael Zanetti on 2013-03-20

make it even more weird

445. By Andrea Cimitan on 2013-03-20

Add test for itemSelected, added readonly property rowWidth for the wdth of the row inside the DashBar, made some properties readonly

444. By Andrea Cimitan on 2013-03-19

Add test for hide after navigation

443. By Andrea Cimitan on 2013-03-19

Add slow swipe

442. By Andrea Cimitan on 2013-03-19

Merged trunk

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2013-03-19 09:16:26 +0000
+++ CMakeLists.txt	2013-03-20 15:46:22 +0000
@@ -32,6 +32,7 @@
 # Tests
 enable_testing()
 add_custom_target(check make test CTEST_OUTPUT_ON_FAILURE=1)
+add_dependencies(check make unittests CTEST_OUTPUT_ON_FAILURE=1)
 
 # Autopilot tests
 include(autopilot)

=== modified file 'Dash/DashBar.qml'
--- Dash/DashBar.qml	2013-01-15 11:37:07 +0000
+++ Dash/DashBar.qml	2013-03-20 15:46:22 +0000
@@ -17,6 +17,7 @@
 import QtQuick 2.0
 import Ubuntu.Components 0.1
 import Unity 0.1
+import "../Components"
 
 Item {
     id: dashBar
@@ -24,23 +25,24 @@
     property var model
     property int currentIndex
 
-    property int lineHeight: units.dp(2)
-    property int itemSize: units.gu(7)
-    property int iconSize: units.gu(2.5)
-    property int initialHeight: lineHeight
-    property int maxHeight: units.gu(6)
+    readonly property alias rowWidth: row.width
+    readonly property int lineHeight: units.dp(2)
+    readonly property int itemSize: units.gu(7)
+    readonly property int iconSize: units.gu(2.5)
+    readonly property int initialHeight: lineHeight
+    readonly property int maxHeight: units.gu(6)
 
     width: units.gu(40)
     height: initialHeight
     Behavior on height {NumberAnimation { duration: 300; easing.type: Easing.OutQuad}}
 
-    state: "initial"
+    state: "hint"
 
     signal itemSelected(int index)
 
     function startNavigation() {
         timeout.stop()
-        dashBar.state = "navigation"
+        dashBar.state = "spread"
     }
 
     function stopNavigation() {
@@ -48,7 +50,140 @@
     }
 
     function finishNavigation() {
-        dashBar.state = "initial"
+        dashBar.state = "hint"
+    }
+
+    states: [
+        State {
+            name: "hint"
+            PropertyChanges {
+                target: bar
+                y: bar.height - initialHeight
+            }
+            PropertyChanges {
+                target: dashBar
+                height: initialHeight
+            }
+        },
+        State {
+            name: "moving"
+            PropertyChanges {
+                target: bar
+                y: MathUtils.clamp(bar.height, draggingArea.mouseY - internal.movingDelta, 0, bar.height)
+            }
+            PropertyChanges {
+                target: dashBar
+                height: Math.max(bar.height - bar.y, initialHeight)
+            }
+        },
+        State {
+            name: "spread"
+            PropertyChanges {
+                target: bar
+                y: 0
+            }
+            PropertyChanges {
+                target: dashBar
+                height: maxHeight
+            }
+        }
+    ]
+
+    transitions: [
+        Transition {
+            to: "hint"
+            PropertyAnimation {
+                target: bar
+                properties: "y"
+                duration: 100
+                easing.type: Easing.OutQuad
+            }
+        },
+        Transition {
+            to: "spread"
+            PropertyAnimation {
+                target: bar
+                properties: "y"
+                duration: 100
+                easing.type: Easing.OutQuad
+            }
+        }
+    ]
+
+    QtObject {
+        id: internal
+        property string previousState: "hint"
+        property int movingDelta
+    }
+
+    onStateChanged: {
+        if (state == "hint" || (state == "moving" && internal.previousState == "hint")) {
+            internal.movingDelta = dashBar.initialHeight + draggingArea.initialY - bar.height;
+        } else if (state == "moving" && internal.previousState == "spread") {
+            internal.movingDelta = draggingArea.initialY;
+        }
+
+        internal.previousState = state;
+    }
+
+    Timer {
+        id: timeout
+        interval: 1500
+        running: false
+        repeat: false
+        onTriggered: finishNavigation()
+    }
+
+    Item {
+        id: bar
+        height: maxHeight
+        anchors {
+            left: parent.left
+            right: parent.right
+        }
+
+        y: dashBar.state == "spread" ? 0 : height - initialHeight
+    }
+
+    DraggingArea {
+        orientation: Qt.Vertical
+        id: draggingArea
+        anchors {
+            bottom: parent.bottom
+            left: parent.left
+            right: parent.right
+        }
+        height: dashBar.state == "spread" ? bar.height + units.gu(1) : units.gu(2)
+        zeroVelocityCounts: true
+        propagateComposedEvents: true
+
+        property int initialY
+
+        onPressed: {
+            if (dashBar.state == "hint")
+                initialY = mouseY
+        }
+
+        onPositionChanged: {
+            dashBar.state = "moving";
+        }
+
+        onReleased: finishMoving()
+        // Mouse cursor moving out of the window while pressed on desktop
+        onCanceled: finishMoving()
+
+        // FIXME: Make all parameters below themable.
+        //  The value of 44 was copied from the Launcher.
+        function finishMoving() {
+            if (draggingArea.dragVelocity < -44) {
+                dashBar.state = "spread";
+            } else if (draggingArea.dragVelocity > 44) {
+                dashBar.state = "hint";
+            } else {
+                dashBar.state = (bar.y < bar.height / 2) ? "spread" : "hint";
+            }
+            timeout.restart()
+        }
     }
 
     Rectangle {
@@ -64,7 +199,7 @@
             anchors.horizontalCenter: parent.horizontalCenter
             currentIndex: dashBar.currentIndex
             onMovingChanged: if (moving) { timeout.stop() } else { timeout.restart() }
-            interactive: visibleArea.widthRatio < 1 && dashBar.state == "navigation"
+            interactive: visibleArea.widthRatio < 1 && dashBar.state == "spread"
             delegate:
                 Item {
                     width: itemSize
@@ -85,22 +220,12 @@
                         smooth: true
                     }
                     MouseArea {
-                        enabled: dashBar.state == "navigation"
+                        enabled: dashBar.state == "spread"
                         anchors.fill: parent
-                        onPressed: {
-                            timeout.stop()
-                            if (dashbar.state == "initial") {
-                                dashbar.state = "navigation"
-                                mouse.accepted = false
-                            }
-                        }
-
-                        onReleased: {
-                            timeout.restart()
-                        }
 
                         onClicked: {
                             dashBar.itemSelected(index)
+                            timeout.restart()
                         }
                     }
                 }
@@ -119,30 +244,5 @@
                 }
         }
     }
-
-    Timer {
-        id: timeout
-        interval: 1500
-        running: false
-        repeat: false
-        onTriggered: finishNavigation()
-    }
-
-    states: [
-        State {
-            name: "initial"
-            PropertyChanges {
-                target: dashBar
-                height: initialHeight
-            }
-        },
-        State {
-            name: "navigation"
-            PropertyChanges {
-                target: dashBar
-                height: maxHeight
-            }
-        }
-    ]
 }
 

=== modified file 'tests/qmluitests/CMakeLists.txt'
--- tests/qmluitests/CMakeLists.txt	2013-03-20 09:39:25 +0000
+++ tests/qmluitests/CMakeLists.txt	2013-03-20 15:46:22 +0000
@@ -20,6 +20,7 @@
     add_dependencies(qmluitests test${COMPONENT_NAME})
 endmacro(add_qml_test)
 
+add_qml_test(DashBar ${CMAKE_SOURCE_DIR}/plugins)
 add_qml_test(DraggingArea)
 add_qml_test(FilterGrid ${CMAKE_BINARY_DIR}/plugins)
 add_qml_test(Hud ${CMAKE_CURRENT_BINARY_DIR}/qml)

=== modified file 'tests/qmluitests/UnityTestCase.qml'
--- tests/qmluitests/UnityTestCase.qml	2013-03-20 14:43:40 +0000
+++ tests/qmluitests/UnityTestCase.qml	2013-03-20 15:46:22 +0000
@@ -26,11 +26,12 @@
     }
 
     // Flickable won't recognise a single mouse move as dragging the flickable.
-    // Use 5 steps because it's what 
+    // Use 5 steps because it's what
     // Qt uses in QQuickViewTestUtil::flick
-    function mouseFlick(item, x, y, toX, toY, pressMouse, releaseMouse) {
-        pressMouse = ((pressMouse != null) ? pressMouse : true); // Default to true for pressMouse if not present
-        releaseMouse = ((releaseMouse != null) ? releaseMouse : true); // Default to true for releaseMouse if not present
+    function mouseFlick(item, x, y, toX, toY, pressMouse, releaseMouse, duration) {
+        if (pressMouse === undefined) pressMouse = true; // Default to true for pressMouse if not present
+        if (releaseMouse === undefined) releaseMouse = true; // Default to true for releaseMouse if not present
+        if (duration === undefined) duration = 0; // Default to 0 for duration if not present
         var nIterations = 5
         var diffX = (toX - x) / nIterations
         var diffY = (toY - y) / nIterations
@@ -40,6 +41,7 @@
         }
         for (var i = 0; i < nIterations; ++i) {
             fakeDateTime.currentTimeMs += 10
+            wait(duration / nIterations)
             if (i === nIterations - 1) {
                 // Avoid any rounding errors by making the last move be at precisely
                 // the point specified

=== added file 'tests/qmluitests/tst_DashBar.qml'
--- tests/qmluitests/tst_DashBar.qml	1970-01-01 00:00:00 +0000
+++ tests/qmluitests/tst_DashBar.qml	2013-03-20 15:46:22 +0000
@@ -0,0 +1,157 @@
+/*
+ * Copyright 2013 Canonical Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import QtQuick 2.0
+import QtTest 1.0
+import Unity 0.1
+import "../../Dash"
+
+Item {
+    width: units.gu(40)
+    height: units.gu(71)
+
+    DashBar {
+        id: dashBar
+
+        anchors {
+            left: parent.left
+            right: parent.right
+            bottom: parent.bottom
+        }
+        model: Lenses {}
+    }
+
+    SignalSpy {
+        id: signalSpy
+        signalName: "itemSelected"
+        target: dashBar
+    }
+
+    UnityTestCase {
+        name: "DashBarTest"
+        when: windowShown
+
+        property alias model: dashBar.model
+        property alias currentIndex: dashBar.currentIndex
+
+        readonly property alias rowWidth: dashBar.rowWidth
+        readonly property alias lineHeight: dashBar.lineHeight
+        readonly property alias itemSize: dashBar.itemSize
+        readonly property alias iconSize: dashBar.iconSize
+        readonly property alias initialHeight: dashBar.initialHeight
+        readonly property alias maxHeight: dashBar.maxHeight
+
+        function checkInitialHeight() {
+            // basically waits for the dashBar to hide... (initial status)
+            tryCompare(dashBar, "height", dashBar.initialHeight)
+        }
+
+        function test_navigationAndHide() {
+            checkInitialHeight()
+
+            dashBar.startNavigation()
+            tryCompare(dashBar, "height", dashBar.maxHeight)
+
+            dashBar.finishNavigation()
+            tryCompare(dashBar, "height", dashBar.initialHeight)
+        }
+
+        function test_bottomSwipeComplete() {
+            checkInitialHeight()
+
+            // (item, x, y, button, modifiers, delay)
+            mousePress(parent,
+                       parent.width / 2, parent.height,
+                       Qt.LeftButton, Qt.NoModifier, 0);
+            // (item, x, y, delay, button)
+            mouseMove(parent,
+                      parent.width / 2, parent.height01,
+                      0, Qt.LeftButton);
+            // (item, x, y, button, modifiers, delay)
+            mouseRelease(parent,
+                         parent.width / 2, parent.height01,
+                         Qt.LeftButton, Qt.NoModifier, 0);
+            tryCompare(dashBar, "height", dashBar.maxHeight)
+        }
+
+        function test_bottomSwipeBack() {
+            checkInitialHeight()
+
+            mousePress(parent,
+                       parent.width / 2, parent.height,
+                       Qt.LeftButton, Qt.NoModifier, 0);
+            mouseMove(parent,
+                      parent.width / 2, parent.height - dashBar.maxHeight / 2,
+                      0, Qt.LeftButton);
+            mouseMove(parent,
+                      parent.width / 2, parent.height,
+                      0, Qt.LeftButton);
+            mouseRelease(parent,
+                         parent.width / 2, parent.height,
+                         Qt.LeftButton, Qt.NoModifier, 0);
+            tryCompare(dashBar, "height", dashBar.initialHeight)
+        }
+
+        function test_bottomSwipeHalfFast() {
+            checkInitialHeight()
+
+            mouseFlick(parent,
+                       parent.width / 2, parent.height,
+                       parent.width / 2, parent.height - 5,
+                       true, true, 0);
+            tryCompare(dashBar, "height", dashBar.maxHeight)
+        }
+
+        function test_bottomSwipeHalfSlow() {
+            checkInitialHeight()
+
+            mouseFlick(parent,
+                       parent.width / 2, parent.height,
+                       parent.width / 2, parent.height - 5,
+                       true, true, 75);
+            tryCompare(dashBar, "height", dashBar.initialHeight)
+        }
+
+        function test_itemSelected() {
+            checkInitialHeight()
+            signalSpy.clear()
+
+            dashBar.startNavigation()
+            tryCompare(dashBar, "height", dashBar.maxHeight)
+
+            var rowStartX = dashBar.width / 2 - rowWidth / 2;
+            for (var i = 0; i < model.rowCount(); i++) {
+                // coordinate x in the middle of item with index 'i'
+                var x = rowStartX + (rowWidth / model.rowCount()) * i + itemSize / 2
+                if (x > rowWidth) {
+                    /* FIXME exit the cycle if we have more lenses than dashBar width.
+                       Let's fix and test this behaviour when we'll have more lenses! */
+                    break;
+                }
+                // FIXME workaround for a bug in SignalSpy
+                signalSpy.clear()
+
+                // (item, x, y, button, modifiers, delay)
+                mouseClick(dashBar, x, dashBar.height / 2, Qt.LeftButton, Qt.NoModifier, 0)
+                compare(signalSpy.count > 0, true, "signal itemSelected not triggered")
+                compare(signalSpy.signalArguments[0][0], i, "signal itemSelected emitted unexpected index");
+            }
+
+            dashBar.finishNavigation()
+        }
+    }
+}
+