Incomplete long-left swipe resets dash
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu UX |
Fix Committed
|
Medium
|
Vesa Rautiainen | ||
unity8 (Ubuntu) |
Fix Released
|
Medium
|
Michael Zanetti |
Bug Description
Steps to reproduce the issue:
1 - open e.g. Ubuntu Store scope (from the app scope) or Youtube scope (from the video scope)
2 - (shortly) swipe from the left to invoke launcher
Expected outcome:
The launcher can be used to open a pinned app (while the current scope is still kept in the background) or dismissed to return to the current scope.
Actual outcome:
The current scope will immediately be closed and the first (main) scope (typically the app scope on devel-images and the Today scope on RTM) will be shown.
If I understand the current design guidelines correctly this is the expected behaviour for a full left-to-right swipe, but not for a short swipe from the left to invoke the launcher.
This behaviour can be observed on both the current devel-proposed and RTM images.
-------
Desired UX
- Don't trigger the "go to home" action yet when the long swipe is started but when user releases the finger and all the long swipe parameters are fulfilled (distance threshold and direction)
- Also trigger "go to home" action when any other scope is focused. Currently action is triggered only when app is focused and when a child scope is focused but not when one of the dash scopes is focused.
Related branches
- Lukáš Tinkl (community): Approve
- Unity8 CI Bot: Approve (continuous-integration)
- Ubuntu Unity PS integration team: Pending requested
-
Diff: 2932 lines (+1624/-416)49 files modifieddata/com.canonical.Unity8.gschema.xml (+2/-2)
debian/control (+3/-1)
plugins/Greeter/Unity/Launcher/CMakeLists.txt (+1/-1)
plugins/Greeter/Unity/Launcher/launcheritem.cpp (+13/-0)
plugins/Greeter/Unity/Launcher/launcheritem.h (+3/-0)
plugins/Unity/Launcher/CMakeLists.txt (+4/-1)
plugins/Unity/Launcher/appdrawermodel.cpp (+62/-0)
plugins/Unity/Launcher/appdrawermodel.h (+33/-0)
plugins/Unity/Launcher/launcheritem.cpp (+13/-0)
plugins/Unity/Launcher/launcheritem.h (+4/-0)
plugins/Unity/Launcher/launchermodel.cpp (+6/-37)
plugins/Unity/Launcher/launchermodel.h (+0/-14)
plugins/Unity/Launcher/plugin.cpp (+2/-1)
plugins/Unity/Launcher/ualwrapper.cpp (+73/-0)
plugins/Unity/Launcher/ualwrapper.h (+35/-0)
plugins/Utils/CMakeLists.txt (+4/-0)
plugins/Utils/appdrawerproxymodel.cpp (+189/-0)
plugins/Utils/appdrawerproxymodel.h (+87/-0)
plugins/Utils/plugin.cpp (+2/-0)
qml/Components/KeyboardShortcutsOverlay.qml (+13/-0)
qml/Launcher/BackgroundBlur.qml (+81/-0)
qml/Launcher/Drawer.qml (+306/-0)
qml/Launcher/DrawerGridView.qml (+47/-0)
qml/Launcher/DrawerListView.qml (+32/-0)
qml/Launcher/Launcher.qml (+150/-30)
qml/Launcher/MoreAppsHeader.qml (+46/-0)
qml/Shell.qml (+10/-34)
qml/Stage/Stage.qml (+3/-6)
qml/Stage/StageMaths.qml (+2/-7)
qml/Tutorial/TutorialLeftLong.qml (+1/-1)
tests/mocks/Unity/Launcher/CMakeLists.txt (+5/-1)
tests/mocks/Unity/Launcher/MockAppDrawerModel.cpp (+75/-0)
tests/mocks/Unity/Launcher/MockAppDrawerModel.h (+32/-0)
tests/mocks/Unity/Launcher/MockLauncherItem.cpp (+13/-0)
tests/mocks/Unity/Launcher/MockLauncherItem.h (+6/-0)
tests/mocks/Unity/Launcher/MockLauncherModel.cpp (+3/-0)
tests/mocks/Unity/Launcher/plugin.cpp (+4/-0)
tests/mocks/Utils/CMakeLists.txt (+4/-0)
tests/mocks/Utils/plugin.cpp (+2/-0)
tests/plugins/Greeter/Unity/Launcher/CMakeLists.txt (+1/-1)
tests/plugins/Unity/Launcher/CMakeLists.txt (+2/-1)
tests/qmltests/CMakeLists.txt (+1/-0)
tests/qmltests/Dash/tst_DashShell.qml (+0/-29)
tests/qmltests/Launcher/tst_Drawer.qml (+245/-0)
tests/qmltests/Launcher/tst_Launcher.qml (+1/-1)
tests/qmltests/Stage/tst_PhoneStage.qml (+0/-26)
tests/qmltests/tst_OrientedShell.qml (+1/-13)
tests/qmltests/tst_Shell.qml (+2/-143)
tests/qmltests/tst_ShellWithPin.qml (+0/-66)
summary: |
- Invoking Launcher while using a Scope exits to Dash/First Scope + Invoking Launcher while using a Scope exits to Dash/First (main) Scope |
description: | updated |
description: | updated |
Changed in unity8 (Ubuntu): | |
assignee: | nobody → Michael Zanetti (mzanetti) |
status: | New → Confirmed |
importance: | Undecided → Medium |
tags: | added: launcher |
Changed in ubuntu-ux: | |
assignee: | nobody → Paty Davila (dizzypaty) |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in unity8 (Ubuntu): | |
status: | Incomplete → Triaged |
Changed in unity8 (Ubuntu): | |
status: | In Progress → Fix Released |
This is likely because your short-swipe actually triggers the long-swipe behaviour, but we should undo the change if long-swipe did not commit.