Merge lp://staging/~seb128/unity-control-center/define-unity-env into lp://staging/unity-control-center

Proposed by Sebastien Bacher
Status: Merged
Approved by: Iain Lane
Approved revision: 12775
Merged at revision: 12775
Proposed branch: lp://staging/~seb128/unity-control-center/define-unity-env
Merge into: lp://staging/unity-control-center
Diff against target: 10 lines (+1/-0)
1 file modified
shell/control-center.c (+1/-0)
To merge this branch: bzr merge lp://staging/~seb128/unity-control-center/define-unity-env
Reviewer Review Type Date Requested Status
Iain Lane (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Unity Control Center development team Pending
Review via email: mp+215153@code.staging.launchpad.net

Commit message

shell: set XDG_CURRENT_DESKTOP to Unity so the correct list of panels is
included in the grid. The gnome-menus api respects OnlyShowIn, without that
we end up missing most of the panels in other environment (e.g ubiquity).
We can't change the .desktop keys since that would lead to wrong behaviour
in e.g the Unity dash

Description of the change

shell: set XDG_CURRENT_DESKTOP to Unity so the correct list of panels is
included in the grid. The gnome-menus api respects OnlyShowIn, without that
we end up missing most of the panels in other environment (e.g ubiquity).
We can't change the .desktop keys since that would lead to wrong behaviour
in e.g the Unity dash

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Iain Lane (laney) wrote :

Okay, so there's no way to ignore OnlyShowIn? This is fine then.

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

> Okay, so there's no way to ignore OnlyShowIn? This is fine then.

not that I found no :/ we could probably use another parser than gnome-menus but it feels like like for such changes this cycle

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