Photos in scope not visible until all loaded

Bug #1384393 reported by Kyle Nitzsche
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Unassigned
The Savilerow project
Fix Released
Critical
Unassigned
unity-api (Ubuntu)
Invalid
Critical
Unassigned
unity-scopes-api (Ubuntu)
Invalid
Critical
Unassigned
unity-scopes-shell (Ubuntu)
Invalid
Critical
Unassigned
unity8 (Ubuntu)
Fix Released
Critical
Albert Astals Cid
unity8 (Ubuntu RTM)
Fix Released
Critical
Michał Sawicz

Bug Description

I think this is a generic unity8 issue that is exposed in the My Photos scope. This scope displays photos from the photo roll through mediascanner.

Problem:
When there are many photos, the scope does not display thumbnails for quite a while. (It may even be that no photo thumbnails display until all are loaded into memory.) This is an awkward user experience because the scope looks like nothing is happening for significant periods of time.

Expectation:
Photo thumbnails would display as they arrive. (Perhaps there would be some kind of window around what is currently visible in the scope so that as soon as photos in the current scroll area, before it by some amount, and after it by some amount are in memory, they could display and the user could scroll somewhat, with the window moving appropriately.)

Tags: ota-1 rtm14

Related branches

Changed in unity8 (Ubuntu):
status: New → Confirmed
Changed in savilerow:
status: New → Confirmed
importance: Undecided → Critical
Changed in unity8 (Ubuntu):
importance: Undecided → Critical
tags: added: rtm14
Andrea Cimitan (cimi)
Changed in unity8 (Ubuntu):
status: Confirmed → New
Changed in unity-api (Ubuntu):
importance: Undecided → Critical
Changed in unity-scopes-api (Ubuntu):
importance: Undecided → Critical
Changed in unity-scopes-shell (Ubuntu):
importance: Undecided → Critical
Revision history for this message
Andrea Cimitan (cimi) wrote :

Probably not a bug in unity8, as we simply display what we receive from the backend. From a quick chat, when we receive data we display it, so if you see a blank dash is because the backend didn't set the data yet.

I think the data should behave as a list, where as soon as results are populating they are being appended in batches... for example for 1000 photos, send the first dozen then another dozen etc etc... until you reach 1000. That way we don't have to wait until 1000 photos are being processed in order to be sent to unity8.

Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
status: New → Triaged
assignee: nobody → Albert Astals Cid (aacid)
Revision history for this message
Michał Sawicz (saviq) wrote :

> Probably not a bug in unity8, as we simply display what we receive from
> the backend. From a quick chat, when we receive data we display it, so
> if you see a blank dash is because the backend didn't set the data yet.
>
> I think the data should behave as a list, where as soon as results are
> populating they are being appended in batches... for example for 1000
> photos, send the first dozen then another dozen etc etc... until you
> reach 1000. That way we don't have to wait until 1000 photos are being
> processed in order to be sent to unity8.

Which is exactly what happens. It's not about the *items* not being shown in the UI, just about the images themselves not showing up. The dash has all the data it needs by that time.

Changed in unity-scopes-api (Ubuntu):
status: New → Invalid
Changed in unity-scopes-shell (Ubuntu):
status: New → Invalid
Changed in unity-api (Ubuntu):
status: New → Invalid
kevin gunn (kgunn72)
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → Critical
status: New → Triaged
Changed in unity8 (Ubuntu RTM):
status: Triaged → In Progress
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
kevin gunn (kgunn72)
tags: added: touch-2014-11-13
Revision history for this message
Olli Ries (ories) wrote :

this bug needs to be targeted after RTM, via ota

tags: added: ota-1
removed: touch-2014-11-13
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

not sure about that, it basically freezes the phone for a few seconds

Olli Ries (ories)
Changed in canonical-devices-system-image:
assignee: nobody → Canonical Devices Products (canonical-devices-products-team)
importance: Undecided → High
milestone: none → r1
status: New → Confirmed
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
status: In Progress → New
assignee: Albert Astals Cid (aacid) → nobody
status: New → Triaged
milestone: none → 14.09-ota-1
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
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Michał Sawicz (saviq)
status: Triaged → In Progress
Changed in canonical-devices-system-image:
status: Confirmed → In Progress
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
milestone: 14.09-ota-1 → 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.20150113~rtm-0ubuntu1

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

  [ Albert Astals ]
  * Use QImageReader not to load the image into memory twice
  * Don't create the whole current scope delegates, just height * 3 (LP:
    #1384393)
  * Make CroppedImageSizer async
  * Rework how we set the ranges so we get some more asynchronousity
    from item views
  * Make sure changing a scope doesn't trigger creation/destruction of
    delegates until it's finished

  [ Michael Terry ]
  * Don't block handling power events on loading the greeter's qml and the
    background image. (LP: #1383277)
 -- Ubuntu daily release <email address hidden> Tue, 13 Jan 2015 15:18:13 +0000

Changed in unity8 (Ubuntu RTM):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: In Progress → Fix Released
Changed in savilerow:
assignee: nobody → Yuan-Chen Cheng (ycheng-twn)
Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

hmm, what I saw is the other bug. mark fix commit in Savilerow.

Changed in savilerow:
status: Confirmed → Fix Committed
assignee: Yuan-Chen Cheng (ycheng-twn) → nobody
status: Fix Committed → Fix Released
Revision history for this message
John McAleely (john.mcaleely) wrote :

@pat, is there a reason this ended up in the phondations bucket? Doesn't look like it belongs.

Changed in canonical-devices-system-image:
assignee: Canonical Devices Products (canonical-devices-products-team) → Pat McGowan (pat-mcgowan)
assignee: Pat McGowan (pat-mcgowan) → nobody
Revision history for this message
John McAleely (john.mcaleely) wrote :

I take back the assignment. there appears to be no work to do here :-)

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

Other bug subscribers

Remote bug watches

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