Ubuntu
unity-api package

Need to remove horizontal swiping between previews

Bug #1541967 reported by Michał Sawicz
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Michał Sawicz
Canonical System Image 10 "OTA10"
Ubuntu UX
Fix Committed
Undecided
Unassigned
unity-api (Ubuntu)
Fix Released
Medium
Paweł Stołowski
unity-scopes-shell (Ubuntu)
Fix Released
Medium
Paweł Stołowski
unity8 (Ubuntu)
Fix Released
Medium
Andrea Cimitan

Bug Description

When you go to a preview, you're actually getting a list that you can swipe through horizontally, we need to get rid of that, as conflicts with edges and is not discoverable.

See https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1567685/comments/9 for some clarification.

ProblemType: Bug
DistroRelease: Ubuntu 15.04
Package: unity8 8.11+15.04.20160122-0ubuntu1
Uname: Linux 3.4.67 armv7l
ApportVersion: 2.17.2-0ubuntu1.3touch1
Architecture: armhf
Date: Thu Feb 4 19:09:01 2016
InstallationDate: Installed on 2016-01-30 (5 days ago)
InstallationMedia: Ubuntu 15.04 - armhf (20160130-020304)
SourcePackage: unity8
UpgradeStatus: No upgrade log present (probably fresh install)

See original description

Related branches

lp:~stolowski/unity-api/single-preview
Albert Astals Cid (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Unity8 CI Bot: Needs Fixing (continuous-integration)
Diff: 427 lines (+16/-283)
10 files modified
debian/changelog (+7/-0)
include/unity/shell/scopes/PreviewStackInterface.h (+0/-99)
include/unity/shell/scopes/ScopeInterface.h (+3/-3)
test/qmltest/mocks/plugins/Unity/Scopes/CMakeLists.txt (+0/-2)
test/qmltest/mocks/plugins/Unity/Scopes/Mocks/MockPreviewStack.cpp (+0/-63)
test/qmltest/mocks/plugins/Unity/Scopes/Mocks/MockPreviewStack.h (+0/-47)
test/qmltest/mocks/plugins/Unity/Scopes/Mocks/MockScope.cpp (+3/-3)
test/qmltest/mocks/plugins/Unity/Scopes/Mocks/MockScope.h (+1/-1)
test/qmltest/mocks/plugins/Unity/Scopes/TestScopesPlugin.cpp (+1/-3)
test/qmltest/unity/shell/scopes/tst_Scopes.qml (+1/-62)
lp:~stolowski/unity-scopes-shell/single-preview
Albert Astals Cid (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Diff: 1350 lines (+275/-516)
22 files modified
debian/HARNESS_VERSION (+1/-1)
debian/control.in (+1/-1)
debian/libscope-harness.symbols.in (+1/-2)
po/POTFILES.in (+3/-2)
src/Unity/CMakeLists.txt (+0/-2)
src/Unity/collectors.h (+1/-1)
src/Unity/plugin.cpp (+1/-3)
src/Unity/previewmodel.cpp (+199/-0)
src/Unity/previewmodel.h (+22/-0)
src/Unity/previewstack.cpp (+0/-338)
src/Unity/previewstack.h (+0/-94)
src/Unity/scope.cpp (+15/-14)
src/Unity/scope.h (+4/-4)
src/scope-harness/internal/preview-widget-arguments.h (+0/-3)
src/scope-harness/preview/preview-widget.cpp (+5/-9)
src/scope-harness/results/result.cpp (+5/-5)
src/scope-harness/test-utils.cpp (+0/-13)
src/scope-harness/test-utils.h (+0/-3)
src/scope-harness/view/preview-view.cpp (+15/-17)
src/scope-harness/view/preview-view.h (+2/-2)
tests/overviewtest.cpp (+0/-1)
tests/previewtest.cpp (+0/-1)
lp:~cimi/unity8/single-preview
Albert Astals Cid (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Unity8 CI Bot: Needs Fixing (continuous-integration)
Diff: 917 lines (+93/-360)
19 files modified
debian/control (+1/-1)
qml/Dash/GenericScopeView.qml (+13/-41)
qml/Dash/PreviewView.qml (+15/-79)
qml/Dash/Previews/Preview.qml (+0/-4)
qml/Dash/Previews/PreviewExpandable.qml (+0/-1)
qml/Dash/Previews/PreviewRatingDisplay.qml (+0/-1)
qml/Dash/Previews/PreviewRatingEdit.qml (+0/-1)
qml/Dash/Previews/PreviewWidget.qml (+0/-3)
qml/Dash/Previews/PreviewWidgetFactory.qml (+0/-4)
tests/mocks/Unity/CMakeLists.txt (+0/-2)
tests/mocks/Unity/Unity.qmltypes (+33/-50)
tests/mocks/Unity/fake_previewstack.cpp (+0/-68)
tests/mocks/Unity/fake_previewstack.h (+0/-52)
tests/mocks/Unity/fake_scope.cpp (+2/-2)
tests/mocks/Unity/fake_scope.h (+2/-2)
tests/mocks/Unity/fake_unity_plugin.cpp (+1/-2)
tests/qmltests/CMakeLists.txt (+1/-1)
tests/qmltests/Dash/tst_GenericScopeView.qml (+0/-28)
tests/qmltests/Dash/tst_PreviewView.qml (+25/-18)
Revision history for this message
Michał Sawicz (saviq) wrote :
Changed in ubuntu-ux:
status: New → Fix Committed
Andrea Cimitan (cimi)
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
milestone: none → ww08-2016
assignee: nobody → Michał Sawicz (saviq)
importance: Undecided → Medium
status: New → In Progress
Paweł Stołowski (stolowski)
Changed in unity-scopes-shell (Ubuntu):
assignee: nobody → Pawel Stolowski (stolowski)
Changed in unity-api (Ubuntu):
assignee: nobody → Pawel Stolowski (stolowski)
Changed in unity-scopes-shell (Ubuntu):
status: New → Confirmed
Changed in unity-api (Ubuntu):
status: New → Confirmed
importance: Undecided → Medium
Changed in unity-scopes-shell (Ubuntu):
importance: Undecided → Medium
Paweł Stołowski (stolowski)
Changed in unity-api (Ubuntu):
status: Confirmed → In Progress
Changed in unity-scopes-shell (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.11+16.04.20160301-0ubuntu1

---------------
unity8 (8.11+16.04.20160301-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Hook the new scope.activationInProgress property (LP: #1537132)

  [ Andrea Cimitan ]
  * Updated the Preview view to use a single PreviewModel instead old
    PreviewStack, using a Loader instead ListView (LP: #1541967)

  [ CI Train Bot ]
  * Resync trunk.
  * Update translation template

  [ Michał Sawicz ]
  * Add autopilot as DEP-8 tests added: debian/tests/autopilot.sh

 -- Pawel Stolowski <email address hidden> Tue, 01 Mar 2016 10:28:52 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in unity-scopes-shell (Ubuntu):
status: In Progress → Fix Released
Changed in unity-api (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Revision history for this message
Mateo Salta (mateo-salta) wrote :

can we place this as optional enabled, or put it back i was using this to swipe through individual articles in scopes.

workflow:
- click on first preview in say a news article list
- scan article
- swipe to next
- channel surf through articles

new behaviour:
- click on article
- reach for back button
- look for second article
- click on it
- whops wrong article
- reach for back button
- read through titles of articles
-click on one
- didnt want to read that one
- reach for back button
- click on next
-read all of that one
- reach for back button...
- give up

Revision history for this message
Felix Wilke (felixw83) wrote :

I really would like to have the old behavior back. I used the feature on a daily basis. Never experiences any issues with edges! Swiping at the center of the screen allowed for browsing through a list.

A usual use case was: search for an appropriate app:
search term e.g. "Recorder" in software center
Access one item, and look at the description and voting of others by swiping.

Michał Sawicz (saviq)
description: updated
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
Revision history for this message
baj (benjamin-vallin) wrote :

As Felix & Mateo, I would like to see the old approach back (as an optional feature?)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Related questions

Remote bug watches

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