Unity

Dash doesn't hide when clicking an app in the "Available" group

Bug #612537 reported by Mikkel Kamstrup Erlandsen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Unity
Fix Released
High
Mikkel Kamstrup Erlandsen
Unity 2010-08-26 "0.2.30"
unity-lens-applications
Fix Released
High
Mikkel Kamstrup Erlandsen
unity (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

In the applications place, when clicking an Available app (loading up the software-center) the dash doesn't hide. This means the user is unable to see the software center.

Note to implementer: The files place and apps place have different behaviour here. The files place activates folder browsing which should not hide the dash. Maybe a new key in the [Activation] group of the .place file to control the behaviour?

Related branches

lp:~unity-team/unity/updated-activation-api
Neil J. Patel (community): Approve
Diff: 204 lines (+102/-65)
2 files modified
unity-private/places/places-place.vala (+87/-3)
unity-private/places/places-view.vala (+15/-62)
Hernando Torque (htorque)
Changed in unity:
status: New → Confirmed
Neil J. Patel (njpatel)
Changed in unity:
assignee: nobody → Neil J. Patel (njpatel)
importance: Undecided → High
milestone: none → 2010-08-26
status: Confirmed → Triaged
Revision history for this message
Neil J. Patel (njpatel) wrote :

New key sounds good. Can you choose a name for it and add it to the spec and the daemons, and I'll implement it on view-side for this week too.

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Giving this a bit of thought...

While a new key in the .place files is nice because it preserves our DBus API it also means that a place daemon can only offer one type of behaviour - which I think is generally a bad solution. So very conservative proposal would be:

 - For Maverick: Use a key in the .place file

 - For Natty: Change return signature of the Activate() DBus call from 'b' to 'u'. Where this 'u' can be one of:
  0: Not launched
  1: Launched, don't hide dash
  2: Launched, hide dash
  N: What ever we need in the future

Or is this API change small enough to get into Maverick? Making sure that place daemons dep on the latest libunity would make upgrades clean at least.

Mikkel Kamstrup Erlandsen (kamstrup)
Changed in unity-place-applications:
assignee: nobody → Mikkel Kamstrup Erlandsen (kamstrup)
importance: Undecided → High
status: New → Triaged
Mikkel Kamstrup Erlandsen (kamstrup)
Changed in unity-place-applications:
status: Triaged → Fix Committed
Mikkel Kamstrup Erlandsen (kamstrup)
Changed in unity:
assignee: Neil J. Patel (njpatel) → Mikkel Kamstrup Erlandsen (kamstrup)
Mikkel Kamstrup Erlandsen (kamstrup)
Changed in unity:
status: Triaged → Fix Committed
Didier Roche-Tolomelli (didrocks)
Changed in unity-place-applications:
status: Fix Committed → Fix Released
Changed in unity:
status: Fix Committed → Fix Released
Marco Trevisan (Treviño) (3v1n0)
Changed in unity (Ubuntu):
status: New → 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.