Merge lp://staging/~khurshid-alam/indicator-appmenu/ignore-desktop into lp://staging/indicator-appmenu
Proposed by
Khurshid Alam
| Status: | Merged |
|---|---|
| Merged at revision: | 285 |
| Proposed branch: | lp://staging/~khurshid-alam/indicator-appmenu/ignore-desktop |
| Merge into: | lp://staging/indicator-appmenu |
| Prerequisite: | lp://staging/~khurshid-alam/indicator-appmenu/fix-build-disco |
| Diff against target: |
125 lines (+11/-17) 5 files modified
src/gdk-get-func.c (+3/-3) src/indicator-appmenu.c (+4/-4) src/window-menu-dbusmenu.c (+2/-4) src/window-menu-model.c (+2/-4) src/window-menu.c (+0/-2) |
| To merge this branch: | bzr merge lp://staging/~khurshid-alam/indicator-appmenu/ignore-desktop |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Marco Trevisan (Treviño) | Pending | ||
|
Review via email:
|
|||
Commit message
* Don't use the deprecated g_type_
* Use X11 specific error trap functions
* Do not to track menus for desktop_windows. Fixes LP: #1827411
Description of the change
Bamf somehow not considering desktop window as valid bamf window (see https:/
To post a comment you must log in.
Revision history for this message
| Khurshid Alam (khurshid-alam) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I figured out what is happening. Before nautilus & nautilus-desktop was the same binary (same for nemo), bamf used to track nautilus for desktop menu. But after the split, the code was never adapted in bamf. But for now, this works.