Merge lp://qastaging/~fboucault/unity-2d/launcher_kde3_support into lp://qastaging/unity-2d/3.0

Proposed by Florian Boucault
Status: Merged
Approved by: Ugo Riboni
Approved revision: 625
Merged at revision: 629
Proposed branch: lp://qastaging/~fboucault/unity-2d/launcher_kde3_support
Merge into: lp://qastaging/unity-2d/3.0
Diff against target: 44 lines (+21/-0)
2 files modified
libunity-2d-private/Unity2d/launcherapplicationslist.cpp (+20/-0)
libunity-2d-private/Unity2d/launcherapplicationslist.h (+1/-0)
To merge this branch: bzr merge lp://qastaging/~fboucault/unity-2d/launcher_kde3_support
Reviewer Review Type Date Requested Status
Ugo Riboni (community) Approve
Review via email: mp+67856@code.qastaging.launchpad.net

Description of the change

[launcher] KDE3 applications now appear in the launcher.

That required to take into account the case where a BamfApplication's user_visible
went from false to true.

To post a comment you must log in.
625. By Florian Boucault

Empty commit to link bugs fixed by that branch.

Revision history for this message
Ugo Riboni (uriboni) wrote :

Fixes the problem as advertised.
Code looks good too (use of UniqueConnection is clever)

review: Approve

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