Merge lp:~3v1n0/indicator-sound/launch-context into lp:indicator-sound/13.10
Proposed by
Marco Trevisan (Treviño)
Status: | Merged |
---|---|
Approved by: | Lars Karlitski |
Approved revision: | 345 |
Merged at revision: | 345 |
Proposed branch: | lp:~3v1n0/indicator-sound/launch-context |
Merge into: | lp:indicator-sound/13.10 |
Diff against target: |
105 lines (+18/-8) 6 files modified
src/Makefile.am (+1/-0) src/metadata-menu-item.vala (+6/-3) src/mpris2-controller.vala (+1/-1) src/player-controller.vala (+5/-2) src/sound-service.c (+1/-1) src/transport-menu-item.vala (+4/-1) |
To merge this branch: | bzr merge lp:~3v1n0/indicator-sound/launch-context |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Lars Karlitski (community) | Approve | ||
Review via email: mp+157356@code.launchpad.net |
Commit message
MenuItem: use GtkLaunchContext with proper event timeout to launch applications
Also discards non-activation events
Description of the change
Like all the launcher applications, also indicator-sound should properly use the GAppLaunchContext when launching an application. Changed the code to use the event timestamp of the click action with the GtkAppLaunchCon
This will allow to make indicator-sound to work nicely with the compiz focus prevention mechanism.
In handle_event calls we also need to ignore the non-activation events.
Upstream port of lp:~3v1n0/indicator-sound/launch-context/+merge/156918
To post a comment you must log in.
Approved, same as the one merged in 13.04.