Ubuntu
unity8 package

[Apps switcher] Visual feedback of user's finger movement needed throughout the gesture

Bug #1368668 reported by Vesa Rautiainen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Unassigned
Canonical System Image ww05-2015
Ubuntu UX
Fix Released
High
Vesa Rautiainen
unity8 (Ubuntu)
Fix Released
Medium
Michael Zanetti
unity8 (Ubuntu RTM)
Fix Released
Medium
Michał Sawicz
Ubuntu RTM 14.09-ota-2

Bug Description

Right edge compound gesture has two phases. During the second phase all the recent applications stack together in to the apps spread. There is a second phase commit threshold value after which this stacking happens. Currently when this commit threshold has been exceeded user loses control over the gesture. There is no way to go back and the UI doesn't react to further finger movement to left. We want user to be in control also after the second commit line.

RIght edge gesture progress is explained and broken into pieces in this spec:
https://docs.google.com/a/canonical.com/document/d/1FC_-5yz5kPy_ZvTphaqxOgI5BPnALFbxRAO4eJkzlf8/edit#

There is already a bug about reversibility (movement back to right) after second commit line (#1355284). So this bug is to fix the non-responsiveness after second phase commit line when user moves finger towards left edge.

Steps to repro:
1. Start right edge swipe
2. keep finger pressed and drag to left until you reach the point when apps spread is formed
3. keep still finger pressed and move finger towards left

Current behaviour:
No visual feedback

Expected behaviour:
Applications should start moving to left and stack the way they do when they're dragged to left in normal spread use case

Tags: ota-1

Related branches

lp://qastaging/~mzanetti/unity8/reversible-spread
Michał Sawicz: Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Albert Astals Cid (community): Approve
Diff: 233 lines (+31/-51)
4 files modified
qml/Stages/PhoneStage.qml (+17/-33)
qml/Stages/TabletStage.qml (+5/-14)
tests/autopilot/unity8/application_lifecycle/tests/test_application_lifecycle.py (+1/-1)
tests/qmltests/Stages/tst_PhoneStage.qml (+8/-3)
lp://qastaging/ubuntu/vivid/unity8
lp://qastaging/~unity-team/unity8/rtm-20150113
Michael Zanetti (community): Approve
Diff: 298 lines (+63/-55)
7 files modified
debian/changelog (+16/-2)
qml/Dash/Dash.qml (+1/-2)
qml/Stages/PhoneStage.qml (+17/-33)
qml/Stages/TabletStage.qml (+5/-14)
tests/autopilot/unity8/application_lifecycle/tests/test_application_lifecycle.py (+1/-1)
tests/qmltests/Dash/tst_Dash.qml (+17/-0)
tests/qmltests/Stages/tst_PhoneStage.qml (+6/-3)
lp://qastaging/~ci-train-bot/unity8/ubuntu-rtm-14.09-proposed
Vesa Rautiainen (vesar)
Changed in ubuntu-ux:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Vesa Rautiainen (vesar)
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

not rtm blocker

Vesa Rautiainen (vesar)
tags: added: ota-1
Changed in ubuntu-ux:
importance: Medium → High
kevin gunn (kgunn72)
Changed in unity8 (Ubuntu):
importance: Undecided → Medium
kevin gunn (kgunn72)
Changed in unity8 (Ubuntu):
assignee: nobody → Andrea Cimitan (cimi)
status: New → Triaged
Changed in unity8 (Ubuntu RTM):
status: New → Triaged
importance: Undecided → Medium
kevin gunn (kgunn72)
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Andrea Cimitan (cimi)
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
assignee: Andrea Cimitan (cimi) → Michael Zanetti (mzanetti)
Changed in unity8 (Ubuntu RTM):
assignee: Andrea Cimitan (cimi) → Michael Zanetti (mzanetti)
Michael Zanetti (mzanetti)
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
Changed in unity8 (Ubuntu RTM):
assignee: Michael Zanetti (mzanetti) → Michał Sawicz (saviq)
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
assignee: Michał Sawicz (saviq) → nobody
status: Triaged → New
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.01+15.04.20141202-0ubuntu1

---------------
unity8 (8.01+15.04.20141202-0ubuntu1) vivid; urgency=low

  [ Michael Terry ]
  * Make sure that there is no window of opportunity for swiping away
    greeter before the lockscreen appears.
  * Don't allow dragging the greeter out from the left edge if it's
    hidden there. (LP: #1372952)

  [ Michał Sawicz ]
  * Move from Theme colour to direct UbuntuColors.orange for activity
    indicator and preview buttons

  [ Albert Astals ]
  * Use QImageReader not to load the image into memory twice (LP:
    #1384374)
  * Don't create the whole current scope delegates, just height * 3 (LP:
    #1384393, #1384374)
  * Make CroppedImageSizer async
  * Use ubuntu::connectivity::NetworkingStatus instead of
    QNetworkConfigurationManager
  * Compile with Qt 5.4 (LP: #1395636)

  [ Michael Zanetti ]
  * close previews and temp scopes on left edge/BFB (LP: #1391798)
  * make spread reversible (LP: #1368668, #1355284, #1368677)

  [ Andrea Cimitan ]
  * Different drag behaviour for not closable apps in spread (LP:
    #1368287)

  [ Daniel d'Andrada ]
  * Make UnityTestCase.touchFlick properly map event coordinates
 -- Ubuntu daily release <email address hidden> Tue, 02 Dec 2014 09:26:49 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Michał Sawicz (saviq) wrote :

This is intertwined with the fix for bug #1355284, please accept for RTM.

Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
importance: Undecided → Medium
milestone: none → ww05-2015
status: New → Confirmed
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Michał Sawicz (saviq)
status: New → In Progress
milestone: none → 14.09-ota-2
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.02+15.04.20150120~rtm-0ubuntu1

---------------
unity8 (8.02+15.04.20150120~rtm-0ubuntu1) 14.09; urgency=medium

  [ Albert Astals ]
  * Fix going to scopes when the Manage Dash is open (LP: #1403464)

  [ Josh Arenson ]
  * Create a PhysicalKeyMapper to handle all physical/hardware keys (LP:
    #1390393)

  [ Michael Zanetti ]
  * make spread reversible (LP: #1368668, #1355284, #1368677)

  [ Nick Dedekind ]
  * Unhook Lights interface from indicator widgets
 -- Ubuntu daily release <email address hidden> Tue, 20 Jan 2015 11:18:21 +0000

Changed in unity8 (Ubuntu RTM):
status: In Progress → Fix Released
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Confirmed → Fix Released
Revision history for this message
Vesa Rautiainen (vesar) wrote :

The final implementation is not exactly as described in design solution but that's fine. This was discussed with the engineer when implementing this and the current solution is actually better than the proposed one.

Changed in ubuntu-ux:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.