Merge lp:~charlesk/appmenu-gtk/lp-788640 into lp:appmenu-gtk/12.10

Proposed by Charles Kerr
Status: Merged
Approved by: Charles Kerr
Approved revision: 153
Merged at revision: 155
Proposed branch: lp:~charlesk/appmenu-gtk/lp-788640
Merge into: lp:appmenu-gtk/12.10
Diff against target: 11 lines (+1/-0)
1 file modified
src/bridge.c (+1/-0)
To merge this branch: bzr merge lp:~charlesk/appmenu-gtk/lp-788640
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
jenkins (community) continuous-integration Approve
Review via email: mp+104963@code.launchpad.net

Description of the change

wicd-gtk's toolbar has a menu which appmenu-gtk exports to the Global Menu. Users don't think to look there because wicd-gtk has no traditional menubar at all, so the end result is confusing to users.

Adding wicd-gtk to appmenu-gtk's list of apps to not export to the Global Menu is one way to avoid this confusion.

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :
review: Approve (continuous-integration)
Revision history for this message
Lars Karlitski (larsu) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/bridge.c'
--- src/bridge.c 2012-04-02 19:12:37 +0000
+++ src/bridge.c 2012-05-07 18:53:18 +0000
@@ -955,6 +955,7 @@
955 || (g_strcmp0 (prg, "emacs23-lucid") == 0)955 || (g_strcmp0 (prg, "emacs23-lucid") == 0)
956 || (g_strcmp0 (prg, "glade") == 0)956 || (g_strcmp0 (prg, "glade") == 0)
957 || (g_strcmp0 (prg, "gnome-panel") == 0)957 || (g_strcmp0 (prg, "gnome-panel") == 0)
958 || (g_strcmp0 (prg, "wicd-client.py") == 0)
958 || (g_strcmp0 (prg, "Lotus Notes") == 0))959 || (g_strcmp0 (prg, "Lotus Notes") == 0))
959 {960 {
960 return TRUE;961 return TRUE;

Subscribers

People subscribed via source and target branches