Favourite scopes forcefully refreshed when opened from Manage Dash

Bug #1447056 reported by Michał Sawicz on 2015-04-22
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Medium
handsome_feng
Vivid
Undecided
Unassigned

Bug Description

Steps:
* swipe left/right through all favourite scopes to trigger a refresh where needed
* bottom-swipe to Manage Dash
* select one of the favourite scopes

Expected:
* the scope is opened pre-populated

Current:
* the scope is refreshed on open

ProblemType: Bug
DistroRelease: Ubuntu 15.04
Package: unity8 8.02+15.04.20150409.1-0ubuntu1
ProcVersionSignature: Ubuntu 3.19.0-13.13-generic 3.19.3
Uname: Linux 3.19.0-13-generic x86_64
ApportVersion: 2.17.2-0ubuntu1
Architecture: amd64
CurrentDesktop: Unity
Date: Wed Apr 22 11:46:36 2015
SourcePackage: unity8
SystemImageInfo:
 current build number: 0
 device name:
 channel: daily
 last update: Unknown
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Michał Sawicz (saviq) wrote :
handsome_feng (feng-kylin) wrote :

I have a branch for this which linked above,and it works fine. But i'm not sure whether it is the most suitable way, maybe there is a better way for this? Modify the "activate" method in unity-scope-shell ? hope to hear your opinion.

Albert Astals Cid (aacid) wrote :

Pawel can you give handsome_feng a suggestion about this?
https://code.launchpad.net/~feng-kylin/unity8/openPrepopulatedScopeInManageDash/+merge/257616 seems a bit of a workaround of the real issue to me, but if you say not calling root.scope.activate(result); is the right way to go for already favorited scopes I'll review handsome_feng's code.

Paweł Stołowski (stolowski) wrote :

Albert will do, let's discuss this tomorrow.

Paweł Stołowski (stolowski) wrote :

Ok, replied in the MP and approved.

Changed in unity-scopes-shell (Ubuntu):
status: New → Invalid
Changed in unity8 (Ubuntu):
status: New → In Progress

@ Pawel Stolowski

You shall mark a bug as "in progress" only after assigning it to yourself.

Changed in unity8 (Ubuntu):
status: In Progress → New
no longer affects: unity-scopes-shell (Ubuntu)
Changed in unity8 (Ubuntu):
status: New → In Progress
importance: Undecided → Critical
importance: Critical → Medium
assignee: nobody → handsome_feng (feng-kylin)
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.02+15.04.20150511-0ubuntu1

---------------
unity8 (8.02+15.04.20150511-0ubuntu1) vivid; urgency=medium

  [ Albert Astals Cid ]
  * Workarounds for concierge mode.

  [ CI Train Bot ]
  * New rebuild forced.
  * Resync trunk.

unity8 (8.02+15.04.20150505-0ubuntu1) vivid; urgency=medium

  [ Albert Astals Cid ]
  * Make runtests fake a test error if make fails
  * Make the test more stable
  * Use dbus-test-runner instead of dbus-launch

  [ Daniel d'Andrada ]
  * DirectionalDragArea: improvements & API grooming (LP: #1417920)
  * Fix EdgeDragEvaluator when a drag can happen both ways
    (Direction.Horizontal)

  [ Josh Arenson ]
  * Remove panelHeight property as it is unused.

  [ Leo Arias ]
  * Initial clean up of the autopilot tests set up. Removed the touch
    device from the test case. Moved the restart of unity to a fixture.
    Removed the unused DragMixin. Updated the setUpClass to use
    process_helpers. Removed the workaround for bug #1238417, already
    fixed. Use the toolkit helper to set the testability environment
    variable. Fixed the indicators base class that was restarting unity
    twice. (LP: #1238417, #1447206)
  * Use the base class from the toolkit in autopilot tests.

  [ Michael Zanetti ]
  * emit application-stop when we're going down (LP: #1326513)

  [ Michał Sawicz ]
  * UNITY_SCOPES_LIST is no more

  [ handsome_feng<email address hidden> ]
  * When click the favorite scope in Dash Manager , it just return to
    the corresponding scope page. (LP: #1447056)

unity8 (8.02+15.04.20150422-0ubuntu1) vivid; urgency=medium

  [ Albert Astals Cid ]
  * Compile with Qt 5.5 (LP: #1437238)
  * Different way of top aligning labels when the other one in the row
    is multiline (LP: #1442085)
  * make pot_file

  [ Andrea Cimitan ]
  * Set sourceSize for DashBackground.qml Image

  [ CI Train Bot ]
  * New rebuild forced.
  * Resync trunk. added: po/sk.po

  [ Daniel d'Andrada ]
  * Move handling of command line options to a separate class
  * Refactor tst_PhysicalKeysMapper.qml

  [ Leo Arias ]
  * For autopilot tests, use the device simulation scenarios from the
    toolkit.

  [ Leonardo Arias Fonseca ]
  * For autopilot tests, use the device simulation scenarios from the
    toolkit.

  [ Michael Zanetti ]
  * Make sure dnd mode is ended properly when drag gesture is cancelled
    (LP: #1444949)

  [ Michał Sawicz ]
  * Fix flake8 warnings (LP: #1444170)
  * Move mock indicator service to unity8-fake-env, as it's a binary-
    dependent package.

  [ Nick Dedekind ]
  * Use asynchronous dbus requests for property updates. (LP: #1436982)

 -- CI Train Bot <email address hidden> Mon, 11 May 2015 08:29:58 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers