Merge lp:~mterry/indicator-appmenu/emit-show-now-changed into lp:indicator-appmenu/0.3
Proposed by
Michael Terry
Status: | Merged |
---|---|
Merged at revision: | 108 |
Proposed branch: | lp:~mterry/indicator-appmenu/emit-show-now-changed |
Merge into: | lp:indicator-appmenu/0.3 |
Diff against target: |
176 lines (+67/-0) 3 files modified
src/indicator-appmenu.c (+37/-0) src/window-menus.c (+26/-0) src/window-menus.h (+4/-0) |
To merge this branch: | bzr merge lp:~mterry/indicator-appmenu/emit-show-now-changed |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
Review via email: mp+52583@code.launchpad.net |
Description of the change
This branch notices changes in its client window Status property and turns them into emissions of show-now-changed on the IndicatorObject. This is in the service of making Alt presses show the appmenu.
This branch requires [1] to work at all and [2] to be useful.
[1] https:/
[2] https:/
To post a comment you must log in.
On Tue, 2011-03-08 at 17:16 +0000, Michael Terry wrote: menus_get_ status (iapp->default_app) != DBUSMENU_ STATUS_ NORMAL) { status_ changed (iapp->default_app, menus_get_ status (iapp-> default_ app),
> + /* Set up initial state for new entries if needed */
> + if (window_
> + window_
> + window_
> + iapp);
> + }
I think that we need to ensure that the one we're switching from wasn't
setting the status to alert as well. I think that the best way to do
this to track the state and see if it changes as we change menus.
review needsfixing