Unity Videos Lens

Merge lp://qastaging/~mhr3/unity-lens-video/fix-1058097 into lp://qastaging/unity-lens-video/remote-videos-scope-trunk

  1. fix-1058097
  2. Merge into remote-videos
Proposed by Michal Hruby
Status: Merged
Merge reported by: Michal Hruby
Merged at revision: not available
Proposed branch: lp://qastaging/~mhr3/unity-lens-video/fix-1058097
Merge into: lp://qastaging/unity-lens-video/remote-videos-scope-trunk
Diff against target: 43 lines (+9/-7)
1 file modified
src/unity-scope-video-remote (+9/-7)
To merge this branch: bzr merge lp://qastaging/~mhr3/unity-lens-video/fix-1058097
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Paweł Stołowski (community) Approve
David Callé Approve
Review via email: mp+126956@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2012-09-28.

Commit message

Refresh online results periodically, not on every view switch

Description of the change

The online videos category in dash flickers every time user switches from a different lens to video lens. This is caused by unnecessary refresh of the results, which causes their removal and re-adding.

To post a comment you must log in.
Revision history for this message
David Callé (davidc3) wrote :

Works fine for me, but I think you a word in the comment.

review: Approve
Revision history for this message
Paweł Stołowski (stolowski) wrote :

The change indeed improves results for online videos. +1

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://s-jenkins:8080/job/unity-scope-video-remote-mbs-autolanding/1/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-scope-video-remote-mbs-autolanding/./build=pbuilder,distribution=quantal,flavor=amd64/1/console
    FAILURE: http://s-jenkins:8080/job/unity-scope-video-remote-mbs-autolanding/./build=pbuilder,distribution=quantal,flavor=i386/1/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Francis Ginther (fginther) wrote :

The automerge job was migrated to an updated system. It found this outstanding MP, but failed due to a configuration issue on the automerge host. Closer inspection reveals that this MP has already been merged to trunk and is no longer valid.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: