Unity 8

Merge lp:~cimi/unity8/unity8.hud-2_hint-reveal-commit into lp:unity8

unity8.hud-2_hint-reveal-commit
Merge into trunk

Proposed by Andrea Cimitan on 2013-09-25

Status:

Merged

Approved by:

Michał Sawicz on 2013-10-03

Approved revision:

333

Merged at revision:

383

Proposed branch:

lp:~cimi/unity8/unity8.hud-2_hint-reveal-commit

Merge into:

lp:unity8

Diff against target:

524 lines (+189/-43)

9 files modified

Bottombar/Bottombar.qml (+81/-16)
Shell.qml (+2/-0)
plugins/Unity/bottombarvisibilitycommunicatorshell.cpp (+15/-1)
plugins/Unity/bottombarvisibilitycommunicatorshell.h (+6/-0)
tests/autopilot/unity8/shell/__init__.py (+14/-0)
tests/autopilot/unity8/shell/emulators/dash.py (+15/-6)
tests/autopilot/unity8/shell/emulators/hud.py (+8/-9)
tests/autopilot/unity8/shell/emulators/main_window.py (+3/-0)
tests/autopilot/unity8/shell/tests/test_hud.py (+45/-11)

To merge this branch:

bzr merge lp:~cimi/unity8/unity8.hud-2_hint-reveal-commit

Related bugs:

Bug #1224480: [HUD] HUD invocation doesn't follow edge gesture sequence	High	Fix Released
Bug #1224633: HUD icon remains when right-swiping to go to the Apps lens	High	Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot (community)	continuous-integration		Approve on 2013-10-03
Michael Zanetti (community)			Approve on 2013-10-02
Michał Sawicz		2013-09-25	Needs Fixing on 2013-09-25
Review via email: mp+187488@code.launchpad.net

Commit message

Add behaviours to the hud reveal.

Description of the change

Add some extra behaviours for the hud button, following design guidelines.
Requires a branch for ubuntu-ui-toolkit to function properly with the toolbar (not required to run, though)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-25:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1112/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Michał Sawicz (saviq) wrote on 2013-09-25:

tests/autopilot/unity8/shell/tests/test_hud.py: multiple new lines at end of file

review: Needs Fixing

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-25:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1122/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-26:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1148/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-30:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1167/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-30:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1169/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Michael Zanetti (mzanetti) wrote on 2013-10-01:

Is position really a double? Seems like a pixel value to me.

===

243 bool forceHidden() const;
244 + double position() const;
245 void setForceHidden(bool forceHidden);
246 + void setPosition(double position);

Usually we group them like this:

243 bool forceHidden() const;
245 void setForceHidden(bool forceHidden);

244 + double position() const;
246 + void setPosition(double position);

===

Can you move the _drag() method to some common place? maybe the __init__.py or something like that. I don't think copying that all over the place is a good idea.

review: Needs Fixing

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-01:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1193/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-01:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1200/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-02:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1208/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-02:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1211/rebuild

review: Approve (continuous-integration)

Revision history for this message

Michael Zanetti (mzanetti) wrote on 2013-10-02:

lgtm

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-02:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/1219/rebuild

review: Approve (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-02:

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-03:

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) on 2013-10-03:

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

Subscribers

People subscribed via source and target branches

to all changes:

Andrea Cimitan

Mohammad Rakibul Hasan

Unity Team

 === modified file 'Bottombar/Bottombar.qml'
 --- Bottombar/Bottombar.qml	2013-09-12 04:51:31 +0000
 +++ Bottombar/Bottombar.qml	2013-10-02 09:27:24 +0000
@@ -34,15 +34,32 @@
      state: "hidden"
++    function hide() {
++        dismissTimer.stop()
++        bottombar.state = "hidden"
++    }
++
++    onApplicationIsOnForegroundChanged: {
++        if (!applicationIsOnForeground) {
++            hide()
++        }
++    }
++
      onStateChanged: {
          if (state == "hidden") {
              dismissTimer.stop()
++            bottomBarVisibilityCommunicatorShell.forceHidden = false
          } else {
              dismissTimer.restart()
+         }
+     }
      onPreventHidingChanged: {
++        if (!preventHiding) {
++            if (state == "hint" || state == "reveal")
++                hide()
++        }
++
          if (dismissTimer.running) {
              dismissTimer.restart();
+         }
@@ -50,7 +67,7 @@
      Timer {
          id: dismissTimer
--        interval: 5000
++        interval: 1000
          onTriggered: {
              if (!bottombar.preventHiding) {
                  bottombar.state = "hidden"
@@ -103,24 +120,29 @@
      EdgeDragArea {
          id: dragArea
++        objectName: "hudDragArea"
          width: parent.width
          height: distanceThreshold
          anchors.bottom: parent.bottom
--        distanceThreshold: units.gu(1)
++        distanceThreshold: units.gu(8)
          enabled: !theHud.shown && bottombar.enabled && applicationIsOnForeground
          direction: Direction.Upwards
++        maxSilenceTime: 2000
          property int previousStatus: -1
          property real touchStartX: -1
++        readonly property real distanceFromThreshold: (-distance) - distanceThreshold // distance is negative
++        readonly property real revealDistance: units.gu(2)
++        readonly property real commitDistance: units.gu(6)
++        readonly property real commitProgress: MathLocal.clamp(distanceFromThreshold / commitDistance, 0, 1)
++
          onStatusChanged: {
              if (status === DirectionalDragArea.WaitingForTouch) {
                  if (previousStatus == DirectionalDragArea.Recognized) {
                      if (hudButton.mouseOver) {
                          hudButton.clicked()
--                    } else {
--                        bottombar.state = "hidden"
+                     }
+                 }
              } else if (status === DirectionalDragArea.Undecided) {
@@ -128,35 +150,78 @@
                      touchStartX = touchX
+                 }
              } else if (status === DirectionalDragArea.Recognized) {
--                bottombar.state = "shown"
++                bottombar.state = "hint"
+             }
              previousStatus = status
+         }
++
++        onDistanceChanged: {
++            if (status === DirectionalDragArea.Recognized) {
++                if (distanceFromThreshold > commitDistance)
++                    bottombar.state = "shown"
++                else if (distanceFromThreshold > revealDistance)
++                    bottombar.state = "reveal"
++            }
++        }
++    }
++
++    Item {
++        id: dismissArea
++        anchors {
++            top: parent.top
++            left: parent.left
++            right: parent.right
++        }
++        height: parent.height - bottomBarVisibilityCommunicatorShell.position
++
++        MouseArea {
++            anchors.fill: parent
++            enabled: bottombar.state == "shown"
++            onPressed: {
++                bottomBarVisibilityCommunicatorShell.forceHidden = true
++                bottombar.state = "hidden"
++            }
++        }
++
++        InputFilterArea {
++            anchors.fill: parent
++            blockInput: (hudButton.opacity == 1)
++        }
+     }
      MouseArea {
--        anchors.fill: parent
--        enabled: bottombar.state == "shown"
++        anchors {
++            top: dismissArea.bottom
++            left: parent.left
++            right: parent.right
++            bottom: parent.bottom
++        }
++        enabled: bottombar.state == "shown" && bottomBarVisibilityCommunicatorShell.position > 0
          onPressed: {
              bottombar.state = "hidden"
+         }
+     }
--    InputFilterArea {
--        anchors.fill: parent
--        blockInput: (hudButton.opacity == 1)
--    }
--
      states: [
          State {
              name: "hidden"
--            PropertyChanges { target: hudButton; opacity: 0}
--            PropertyChanges { target: hudButton; bottomMargin: units.gu(-1)}
++            PropertyChanges { target: hudButton; opacity: 0 }
++            PropertyChanges { target: hudButton; bottomMargin: units.gu(-2) }
++        },
++        State {
++            name: "hint"
++            extend: "hidden"
++            PropertyChanges { target: hudButton; opacity: 0.5 }
++        },
++        State {
++            name: "reveal"
++            extend: "hint"
++            PropertyChanges { target: hudButton; bottomMargin: units.gu(-2) + units.gu(2) * dragArea.commitProgress }
          },
          State {
              name: "shown"
--            PropertyChanges { target: hudButton; opacity: 1}
--            PropertyChanges { target: hudButton; bottomMargin: units.gu(0)}
++            PropertyChanges { target: hudButton; opacity: 1 }
++            PropertyChanges { target: hudButton; bottomMargin: units.gu(0) }
+         }
+     ]
+ }
 === modified file 'Shell.qml'
 --- Shell.qml	2013-09-30 15:47:18 +0000
 +++ Shell.qml	2013-10-02 09:27:24 +0000
@@ -623,6 +623,7 @@
+         }
          Bottombar {
++            id: bottombar
              theHud: hud
              anchors.fill: parent
              enabled: hud.available
@@ -663,6 +664,7 @@
                  if (shown) {
                      panel.indicators.hide()
                      hud.hide()
++                    bottombar.hide()
+                 }
+             }
+         }
 === modified file 'plugins/Unity/bottombarvisibilitycommunicatorshell.cpp'
 --- plugins/Unity/bottombarvisibilitycommunicatorshell.cpp	2013-06-05 22:03:08 +0000
 +++ plugins/Unity/bottombarvisibilitycommunicatorshell.cpp	2013-10-02 09:27:24 +0000
@@ -17,7 +17,8 @@
  #include "bottombarvisibilitycommunicatorshell.h"
  BottomBarVisibilityCommunicatorShell::BottomBarVisibilityCommunicatorShell()
-- : m_forceHidden(false)
++ : m_forceHidden(false),
++   m_position(0)
+ {
+ }
@@ -26,6 +27,11 @@
      return m_forceHidden;
+ }
++double BottomBarVisibilityCommunicatorShell::position() const
++{
++    return m_position;
++}
++
  void BottomBarVisibilityCommunicatorShell::setForceHidden(bool forceHidden)
+ {
      if (forceHidden != m_forceHidden) {
@@ -33,3 +39,11 @@
          Q_EMIT forceHiddenChanged(forceHidden);
+     }
+ }
++
++void BottomBarVisibilityCommunicatorShell::setPosition(double position)
++{
++    if (position != m_position) {
++        m_position = position;
++        Q_EMIT positionChanged(position);
++    }
++}
 === modified file 'plugins/Unity/bottombarvisibilitycommunicatorshell.h'
 --- plugins/Unity/bottombarvisibilitycommunicatorshell.h	2013-06-05 22:03:08 +0000
 +++ plugins/Unity/bottombarvisibilitycommunicatorshell.h	2013-10-02 09:27:24 +0000
@@ -30,6 +30,7 @@
+ {
      Q_OBJECT
      Q_PROPERTY(bool forceHidden READ forceHidden WRITE setForceHidden NOTIFY forceHiddenChanged)
++    Q_PROPERTY(double position READ position WRITE setPosition NOTIFY positionChanged)
      Q_CLASSINFO("D-Bus Interface", "com.canonical.Shell.BottomBarVisibilityCommunicator")
  public:
@@ -41,13 +42,18 @@
      bool forceHidden() const;
      void setForceHidden(bool forceHidden);
++    double position() const;
++    void setPosition(double position);
++
  Q_SIGNALS:
      void forceHiddenChanged(bool forceHidden);
++    void positionChanged(double position);
  private:
      BottomBarVisibilityCommunicatorShell();
      bool m_forceHidden;
++    double m_position;
  };
  #endif
 === modified file 'tests/autopilot/unity8/shell/__init__.py'
 --- tests/autopilot/unity8/shell/__init__.py	2013-09-13 18:09:42 +0000
 +++ tests/autopilot/unity8/shell/__init__.py	2013-10-02 09:27:24 +0000
@@ -19,6 +19,7 @@
  """unity shell autopilot tests and emulators - sub level package."""
++from time import sleep
  from functools import wraps
  import logging
@@ -46,3 +47,16 @@
          tests_self._qml_mock_enabled = False
          return fn(*args, **kwargs)
      return wrapper
++
++class DragMixin(object):
++    def _drag(self, x1, y1, x2, y2):
++        cur_x = x1
++        cur_y = y1
++        dx = 1.0 * (x2 - x1) / 100
++        dy = 1.0 * (y2 - y1) / 100
++        for i in range(0, 100):
++            self.touch._finger_move(int(cur_x), int(cur_y))
++            sleep(0.002)
++            cur_x += dx
++            cur_y += dy
++        self.touch._finger_move(int(x2), int(y2))
 === modified file 'tests/autopilot/unity8/shell/emulators/dash.py'
 --- tests/autopilot/unity8/shell/emulators/dash.py	2013-08-11 22:16:33 +0000
 +++ tests/autopilot/unity8/shell/emulators/dash.py	2013-10-02 09:27:24 +0000
@@ -18,17 +18,22 @@
+ #
  from unity8.shell.emulators import UnityEmulatorBase
--
++from time import sleep
  class Dash(UnityEmulatorBase):
      """An emulator that understands the Dash."""
      def get_home_applications_grid(self):
--        return self.get_scope('home').select_single(
--            "GenericFilterGrid",
--            objectName="applications.scope"
--        )
++        for i in range(1,10):
++            get_grid = self.get_scope('home').select_single(
++                "GenericFilterGrid",
++                objectName="applications.scope"
++            )
++            if get_grid != None:
++                break
++            sleep(1)
++        return get_grid
      def get_application_icon(self, text):
          """Returns a 'Tile' icon that has the text 'text' from the application
@@ -40,7 +45,11 @@
          """
          app_grid = self.get_home_applications_grid()
--        resp_grid = app_grid.select_single('ResponsiveGridView')
++        for i in range(1,10):
++            resp_grid = app_grid.select_single('ResponsiveGridView')
++            if resp_grid != None:
++                break
++            sleep(1)
          return resp_grid.select_single('Tile', text=text)
      def get_scope(self, scope_name='home'):
 === modified file 'tests/autopilot/unity8/shell/emulators/hud.py'
 --- tests/autopilot/unity8/shell/emulators/hud.py	2013-08-01 16:50:45 +0000
 +++ tests/autopilot/unity8/shell/emulators/hud.py	2013-10-02 09:27:24 +0000
@@ -21,36 +21,35 @@
  from unity8 import get_grid_size
  from unity8.shell.emulators import UnityEmulatorBase
++from unity8.shell import DragMixin
  from autopilot.input import Touch
--
  SwipeCoords = namedtuple('SwipeCoords', 'start_x end_x start_y end_y')
--
--class Hud(UnityEmulatorBase):
++class Hud(UnityEmulatorBase, DragMixin):
      """An emulator that understands the Hud."""
      def show(self):
          """Swipes open the Hud."""
--        touch = Touch.create()
++        self.touch = Touch.create()
          window = self.get_root_instance().select_single('QQuickView')
          hud_show_button = window.select_single("HudButton")
          swipe_coords = self.get_button_swipe_coords(window, hud_show_button)
--        touch.press(swipe_coords.start_x, swipe_coords.start_y)
--        touch._finger_move(swipe_coords.end_x, swipe_coords.end_y)
++        self.touch.press(swipe_coords.start_x, swipe_coords.start_y)
++        self._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.end_y)
          try:
              hud_show_button.opacity.wait_for(1.0)
--            touch.release()
++            self.touch.release()
              self.shown.wait_for(True)
          except AssertionError:
              raise
          finally:
--            if touch._touch_finger is not None:
--                touch.release()
++            if self.touch._touch_finger is not None:
++                self.touch.release()
      def dismiss(self):
          """Closes the open Hud."""
 === modified file 'tests/autopilot/unity8/shell/emulators/main_window.py'
 --- tests/autopilot/unity8/shell/emulators/main_window.py	2013-07-30 13:16:50 +0000
 +++ tests/autopilot/unity8/shell/emulators/main_window.py	2013-10-02 09:27:24 +0000
@@ -58,6 +58,9 @@
      def get_hud_show_button(self):
          return self.app.select_single("HudButton")
++    def get_hud_edge_drag_area(self):
++        return self.app.select_single(objectName="hudDragArea")
++
      def get_dash(self):
          return self.app.select_single(Dash)
 === modified file 'tests/autopilot/unity8/shell/tests/test_hud.py'
 --- tests/autopilot/unity8/shell/tests/test_hud.py	2013-09-05 16:31:52 +0000
 +++ tests/autopilot/unity8/shell/tests/test_hud.py	2013-10-02 09:27:24 +0000
@@ -19,29 +19,30 @@
  from __future__ import absolute_import
--from unity8.shell import with_lightdm_mock
++from unity8.shell import with_lightdm_mock, DragMixin
  from unity8.shell.tests import UnityTestCase, _get_device_emulation_scenarios
  from testtools.matchers import Equals
  from autopilot.matchers import Eventually
--
--class TestHud(UnityTestCase):
++class TestHud(UnityTestCase, DragMixin):
      """Tests the Shell HUD."""
      scenarios = _get_device_emulation_scenarios()
      def test_show_hud_button_appears(self):
--        """Swiping up while an app is active must show the 'show hud' button.
++        """Swiping up while an app is active must show the 'show hud' button, following some behaviours.
             The button must disappear not opening the HUD when releasing the
--           mouse again somewhere on the screen except on the button itself.
++           mouse again somewhere on the screen except on the button itself following a timeout.
++           The button must disappear when touching somewhere on the screen except the button itself.
          """
          self.launch_unity()
          self.main_window.get_greeter().swipe()
          window = self.main_window.get_qml_view()
          hud_show_button = self.main_window.get_hud_show_button()
++        edge_drag_area = self.main_window.get_hud_edge_drag_area()
          hud = self.main_window.get_hud()
          self._launch_test_app_from_app_screen()
@@ -50,16 +51,33 @@
              window,
              hud_show_button
+         )
++        initialBottomMargin = int(hud_show_button.bottomMargin)
++
          self.touch.press(swipe_coords.start_x, swipe_coords.start_y)
          self.addCleanup(self._maybe_release_finger)
--        self.touch._finger_move(swipe_coords.end_x, swipe_coords.end_y + hud_show_button.height);
++        self._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.start_y - int(edge_drag_area.distanceThreshold) - 5)
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.5)))
++        self.assertThat(hud_show_button.bottomMargin, Eventually(Equals(initialBottomMargin)))
++        self._drag(swipe_coords.start_x, swipe_coords.start_y - int(edge_drag_area.distanceThreshold) - 5, swipe_coords.end_x, swipe_coords.start_y - int(edge_drag_area.distanceThreshold) - int(edge_drag_area.commitDistance) - 5)
          self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
++        self.assertThat(hud_show_button.bottomMargin, Eventually(Equals(0.0)))
          self.touch.release();
--        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.0)))
--        self.assertThat(hud.shown, Equals(False))
++        self.assertThat(hud.shown, Equals(False))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.0)))
++
++        self.touch.press(swipe_coords.start_x, swipe_coords.start_y)
++        self._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.end_y - int(hud_show_button.height))
++        self.assertThat(hud.shown, Equals(False))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
++        self.touch.release()
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
++        self.touch.tap(swipe_coords.end_x, swipe_coords.end_y - int(hud_show_button.height))
++        self.assertThat(hud.shown, Equals(False))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.0)))
      def test_show_hud_appears(self):
          """Releasing the touch on the 'show hud' button must display the hud.
++           Test that the hud button stays on screen and tapping it opens the hud.
          """
          self.launch_unity()
@@ -74,14 +92,28 @@
              window,
              hud_show_button
+         )
++
          self.touch.press(swipe_coords.start_x, swipe_coords.start_y)
          self.addCleanup(self._maybe_release_finger)
--        self.touch._finger_move(swipe_coords.end_x, swipe_coords.end_y)
++        self._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.end_y)
++        self.assertThat(hud.shown, Eventually(Equals(False)))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
++        self.touch.release()
++        self.assertThat(hud.shown, Eventually(Equals(True)))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.0)))
++        x, y = hud.get_close_button_coords()
++        self.touch.tap(x, y)
++        self.assertThat(hud.shown, Eventually(Equals(False)))
--        self.assertThat(hud.shown, Eventually(Equals(False)))
++        self.touch.press(swipe_coords.start_x, swipe_coords.start_y)
++        self._drag(swipe_coords.start_x, swipe_coords.start_y, swipe_coords.start_x, swipe_coords.end_y - int(hud_show_button.height))
++        self.assertThat(hud.shown, Equals(False))
          self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
          self.touch.release()
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(1.0)))
++        self.touch.tap(swipe_coords.end_x, swipe_coords.end_y)
          self.assertThat(hud.shown, Eventually(Equals(True)))
++        self.assertThat(hud_show_button.opacity, Eventually(Equals(0.0)))
      def test_hide_hud_click(self):
          """Tapping the close button of the Hud must dismiss it."""
@@ -90,6 +122,7 @@
          hud = self.main_window.get_hud()
          self._launch_test_app_from_app_screen()
++
          hud.show()
          x, y = hud.get_close_button_coords()
@@ -103,10 +136,11 @@
          """
          self.launch_unity()
          self.main_window.get_greeter().swipe()
++        window = self.main_window.get_qml_view()
          hud = self.main_window.get_hud()
--        window = self.main_window.get_qml_view()
          self._launch_test_app_from_app_screen()
++
          hud.show()
          start_x, start_y = hud.get_close_button_coords()