Ubuntu
unity8 package

Carousel displays no metadata, making it impossible to identify tracks/videos

Bug #1294294 reported by Alan Pope 🍺🐧🐱 🦄
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UX
Fix Released
Medium
James Mulholland
unity-scope-mediascanner (Ubuntu)
Invalid
Medium
Unassigned
unity8 (Ubuntu)
Fix Released
Medium
Unity UI Team

Bug Description

Put some music and/or videos onto your device via mtp into ~/Music and/or ~/Videos
Go to the Music or Video scope
Observe music covers or video thumbnails.

Note that with multiple tracks from the same album/artist it's impossible to differentiate them as there is no metadata displayed.

It's less pronounced on video carousel as you may have decent thumbnails which let you identify one video from another. However if you have multiple episodes of a show, it's not easy to tell which is which.

ProblemType: Bug
DistroRelease: Ubuntu 14.04
Package: unity8 7.84+14.04.20140317.2-0ubuntu1
Uname: Linux 3.4.0-5-mako armv7l
ApportVersion: 2.13.3-0ubuntu1
Architecture: armhf
Date: Tue Mar 18 19:05:48 2014
InstallationDate: Installed on 2014-03-18 (0 days ago)
InstallationMedia: Ubuntu Trusty Tahr (development branch) - armhf (20140318.1)
ProcEnviron:
 TERM=linux
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 SHELL=/bin/bash
SourcePackage: unity8
UpgradeStatus: No upgrade log present (probably fresh install)

See original description

Related branches

lp:~unity-team/unity8/new-scopes-clean-to-trunk
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Didier Roche-Tolomelli: Approve
Michał Sawicz: Needs Fixing
Jouni Helminen (community): Approve (design)
Michael Zanetti (community): Approve
Ubuntu Unity PS integration team: Pending (packaging) requested
Diff: 9400 lines (+1608/-5707)
116 files modified
data/unity8.conf (+9/-0)
debian/control (+3/-3)
debian/rules (+0/-5)
debian/scopes-recommends-generator (+0/-27)
plugins/DashViews/listviewwithpageheader.cpp (+14/-7)
qml/Components/ListItems/Header.qml (+2/-14)
qml/Components/OpenEffect.qml (+0/-138)
qml/Components/PageHeader.qml (+27/-2)
qml/Components/RatingStars.qml (+0/-41)
qml/Components/ResponsiveGridView.qml (+0/-1)
qml/Components/SeeMore.qml (+4/-2)
qml/Components/Stage.qml (+13/-3)
qml/Components/Tile.qml (+0/-39)
qml/Components/TileStyle.qml (+0/-81)
qml/Dash/Apps/AppPreview.qml (+0/-167)
qml/Dash/Apps/DashPluginFilterGrid.qml (+0/-47)
qml/Dash/Apps/FrequentlyUsedAppsModel.qml (+0/-110)
qml/Dash/Apps/RunningApplicationTile.qml (+2/-3)
qml/Dash/Card.qml (+5/-3)
qml/Dash/CardCarousel.qml (+69/-0)
qml/Dash/CardFilterGrid.qml (+76/-0)
qml/Dash/CardHeader.qml (+2/-2)
qml/Dash/CardTool.qml (+1/-1)
qml/Dash/Dash.qml (+57/-5)
qml/Dash/DashApps.qml (+4/-19)
qml/Dash/DashCarousel.qml (+0/-47)
qml/Dash/DashContent.qml (+61/-43)
qml/Dash/DashContentOpenEffect.qml (+0/-57)
qml/Dash/DashContentTabBarStyle.qml (+3/-0)
qml/Dash/DashFilterGrid.qml (+0/-65)
qml/Dash/DashHome.qml (+0/-39)
qml/Dash/DashPreview.qml (+0/-208)
qml/Dash/DashPreviewPlaceholder.qml (+0/-25)
qml/Dash/DashRenderer.qml (+5/-3)
qml/Dash/Generic/FlatTileStyle.qml (+0/-79)
qml/Dash/Generic/GenericCarousel.qml (+0/-53)
qml/Dash/Generic/GenericFilterGrid.qml (+0/-46)
qml/Dash/Generic/GenericFilterGridPotrait.qml (+0/-28)
qml/Dash/Generic/GenericPreview.qml (+0/-197)
qml/Dash/Generic/WeatherFilterGrid.qml (+0/-47)
qml/Dash/GenericScopeView.qml (+66/-125)
qml/Dash/Movie/MoviePreview.qml (+0/-71)
qml/Dash/Music/CarouselDelegateMusic.qml (+0/-42)
qml/Dash/Music/MusicCarousel.qml (+0/-34)
qml/Dash/Music/MusicFilterGrid.qml (+0/-48)
qml/Dash/Music/MusicPreview.qml (+0/-203)
qml/Dash/Music/MusicTileStyle.qml (+0/-102)
qml/Dash/PreviewDelegateMapper.qml (+0/-38)
qml/Dash/PreviewListView.qml (+59/-140)
qml/Dash/Previews/Header.qml (+0/-109)
qml/Dash/Previews/Preview.qml (+17/-23)
qml/Dash/Previews/PreviewAudioPlayback.qml (+4/-3)
qml/Dash/Previews/PreviewHeader.qml (+2/-0)
qml/Dash/Previews/PreviewImageGallery.qml (+10/-0)
qml/Dash/Previews/PreviewTextSummary.qml (+4/-2)
qml/Dash/Previews/PreviewWidgetFactory.qml (+1/-1)
qml/Dash/Previews/Reviews.qml (+0/-194)
qml/Dash/ScopeDelegateMapper.qml (+1/-3)
qml/Dash/ScopeItem.qml (+94/-0)
qml/Dash/SearchableResultModel.qml (+0/-44)
qml/Dash/Video/CarouselDelegateVideo.qml (+0/-42)
qml/Dash/Video/VideoCarousel.qml (+0/-34)
qml/Dash/Video/VideoFilterGrid.qml (+0/-47)
qml/Dash/Video/VideoTileStyle.qml (+0/-87)
qml/Launcher/Launcher.qml (+0/-1)
qml/Launcher/LauncherPanel.qml (+0/-1)
qml/ScopeTool.qml (+9/-1)
qml/Shell.qml (+10/-32)
tests/autopilot/unity8/shell/emulators/dash.py (+23/-52)
tests/autopilot/unity8/shell/emulators/main_window.py (+0/-4)
tests/autopilot/unity8/shell/tests/__init__.py (+1/-1)
tests/autopilot/unity8/shell/tests/test_emulators.py (+26/-52)
tests/autopilot/unity8/shell/tests/test_greeter.py (+0/-38)
tests/mocks/Unity/CMakeLists.txt (+4/-4)
tests/mocks/Unity/categoryresults.cpp (+0/-121)
tests/mocks/Unity/categoryresults.h (+0/-68)
tests/mocks/Unity/fake_applications_scope.cpp (+0/-76)
tests/mocks/Unity/fake_applications_scope.h (+0/-33)
tests/mocks/Unity/fake_categories.cpp (+127/-212)
tests/mocks/Unity/fake_categories.h (+28/-32)
tests/mocks/Unity/fake_preview.cpp (+0/-73)
tests/mocks/Unity/fake_preview.h (+0/-65)
tests/mocks/Unity/fake_previewmodel.cpp (+43/-53)
tests/mocks/Unity/fake_previewmodel.h (+24/-25)
tests/mocks/Unity/fake_previewstack.cpp (+76/-0)
tests/mocks/Unity/fake_previewstack.h (+61/-0)
tests/mocks/Unity/fake_previewwidgetmodel.cpp (+86/-0)
tests/mocks/Unity/fake_previewwidgetmodel.h (+56/-0)
tests/mocks/Unity/fake_resultsmodel.cpp (+117/-0)
tests/mocks/Unity/fake_resultsmodel.h (+80/-0)
tests/mocks/Unity/fake_scope.cpp (+20/-120)
tests/mocks/Unity/fake_scope.h (+17/-18)
tests/mocks/Unity/fake_scopes.cpp (+1/-5)
tests/mocks/Unity/fake_unity_plugin.cpp (+10/-13)
tests/qmltests/CMakeLists.txt (+0/-10)
tests/qmltests/Components/tst_OpenEffect.qml (+0/-71)
tests/qmltests/Components/tst_PageHeaderLabel.qml (+1/-1)
tests/qmltests/Components/tst_RatingStars.qml (+0/-62)
tests/qmltests/Components/tst_Tile.qml (+0/-71)
tests/qmltests/Dash/Apps/tst_AppPreview.qml (+0/-276)
tests/qmltests/Dash/Movie/tst_MoviePreview.qml (+0/-86)
tests/qmltests/Dash/Music/tst_MusicPreview.qml (+0/-174)
tests/qmltests/Dash/Previews/tst_Preview.qml (+4/-18)
tests/qmltests/Dash/qml/FakeScopeView.qml (+1/-1)
tests/qmltests/Dash/qml/fake_scopeView1.qml (+1/-1)
tests/qmltests/Dash/qml/fake_scopeView2.qml (+1/-1)
tests/qmltests/Dash/qml/fake_scopeView3.qml (+1/-1)
tests/qmltests/Dash/tst_CardTool.qml (+1/-1)
tests/qmltests/Dash/tst_Dash.qml (+14/-6)
tests/qmltests/Dash/tst_DashContent.qml (+116/-34)
tests/qmltests/Dash/tst_DashPreview.qml (+0/-202)
tests/qmltests/Dash/tst_FilterGrids.qml (+0/-143)
tests/qmltests/Dash/tst_GenericPreview.qml (+0/-125)
tests/qmltests/Dash/tst_GenericScopeView.qml (+24/-246)
tests/qmltests/tst_Shell.qml (+9/-28)
tools/registry-tracker.cpp (+1/-0)
lp:ubuntu/trusty-proposed/unity8
Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :
description: updated
Selene ToyKeeper (toykeeper)
tags: added: flo mako qa-touch r244
Revision history for this message
Michał Sawicz (saviq) wrote :

This is not the case any more with new scopes, which will land in the following day or two.

FWIW, the image above shows an issue with the mediascanner (bug #1237970), not with unity8 itself.

Either way, it's not unity8 that decides what to display, but the respective scopes are. At least with new scopes, that is.

Marking as affecting mediascanner scope, just in case this would still happen to be valid in some way when we land new scopes.

Changed in unity8 (Ubuntu):
status: New → In Progress
assignee: nobody → Unity UI Team (unity-ui-team)
Revision history for this message
Michał Sawicz (saviq) wrote :

Oh and btw, new scopes are currently available in silo 13: ppa:ci-train-ppa-service/landing-013

Alberto Salvia Novella (es20490446e)
Changed in unity-scope-mediascanner (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
Changed in unity8 (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 7.84+14.04.20140324.4-0ubuntu1

---------------
unity8 (7.84+14.04.20140324.4-0ubuntu1) trusty; urgency=low

  [ Michal Hruby ]
  * Change and extend the way non-installed scopes are started with the
    scope-tool.
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)

  [ Gerry Boland ]
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)

  [ Michał Sawicz ]
  * Fix rating input action to always be "rated", not dynamic. Based on
    http://developer.ubuntu.com/api/devel/ubuntu-14.04/cplusplus/unity-
    scopes/previewwidgets.html#rating-input
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)

  [ Kevin Gunn ]
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)

  [ Albert Astals ]
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)
  * LVWPH: cull lost items lost items will be released on the next
    updatePolish cycle but meanwhile don't let them be visible

  [ Daniel d'Andrada ]
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)

  [ Michał Karnicki ]
  * Switch to new scope backend and apply required visual adaptations.
    (LP: #1294294)
 -- Ubuntu daily release <email address hidden> Mon, 24 Mar 2014 16:10:24 +0000

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

Adding ubuntu-ux as the visual design still shows "Songs" as the first (and only carousel) category, which seems wrong unless we randomize the list somehow, but I'm not sure why wouldn't those be "Albums" instead

Revision history for this message
Selene ToyKeeper (toykeeper) wrote :

I can confirm it's fixed, as of image 258 (IIRC) and later. The currently-selected file has its title displayed over the icon. Not ideal, but it's a lot better than it was before.

As for the visual design, it seems nice if there are only a dozen or two songs on the device. For my purposes though, it's pretty useless (5000+ songs on the device).

Revision history for this message
Michał Sawicz (saviq) wrote :

That'd be bug #1237970, which we need design input for.

Daniela Ferrai (dferrai)
Changed in ubuntu-ux:
assignee: nobody → Benjamin Keyser (bjkeyser)
Ruby Hatem (rh-r)
Changed in ubuntu-ux:
assignee: Benjamin Keyser (bjkeyser) → Mike Nagle (mikenagle)
assignee: Mike Nagle (mikenagle) → Ruby Hatem (rh-r)
assignee: Ruby Hatem (rh-r) → Mike Nagle (mikenagle)
Revision history for this message
Mike Nagle (mikenagle) wrote :

The latest designs don't have a songs carousel at the top anymore so marking this as fixed. Thanks.

Changed in ubuntu-ux:
status: New → Fix Committed
assignee: Mike Nagle (mikenagle) → nobody
Revision history for this message
Ruby Hatem (rh-r) wrote :

The carousel is still used in video scopes however. This is on the aggregator home screen with local videos only as you can see on page 4 on: https://docs.google.com/a/canonical.com/document/d/1Y9H3SY2uwbeLZSta9bGdLcyGr-tP0LwHyIUigTnieww/edit

Matthew Paul Thomas (mpt)
Changed in ubuntu-ux:
assignee: nobody → James Mulholland (jamesjosephmulholland)
Matthew Paul Thomas (mpt)
Changed in ubuntu-ux:
importance: Undecided → Medium
Michi Henning (michihenning)
Changed in unity-scope-mediascanner (Ubuntu):
status: Triaged → Invalid
James Mulholland (jamesmulholland)
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.