← Back to team overview

gnome-split-team team mailing list archive

[Branch ~respawneral/gnome-split/mainline] Rev 248: Possible fix for GNOME Split crashing during the

 

------------------------------------------------------------
revno: 248
fixes bug(s): https://launchpad.net/bugs/655696
committer: Guillaume Mazoyer <respawneral@xxxxxxxxx>
branch nick: menu-status-icon
timestamp: Sat 2011-02-12 16:19:43 +0100
message:
  Possible fix for GNOME Split crashing during the
  menu creation of the status icon.
modified:
  src/org/gnome/split/gtk/widget/AreaStatusIcon.java


--
lp:gnome-split
https://code.launchpad.net/~respawneral/gnome-split/mainline

Your team GNOME Split developers is subscribed to branch lp:gnome-split.
To unsubscribe from this branch go to https://code.launchpad.net/~respawneral/gnome-split/mainline/+edit-subscription
=== modified file 'src/org/gnome/split/gtk/widget/AreaStatusIcon.java'
--- src/org/gnome/split/gtk/widget/AreaStatusIcon.java	2011-01-04 12:56:24 +0000
+++ src/org/gnome/split/gtk/widget/AreaStatusIcon.java	2011-02-12 15:19:43 +0000
@@ -82,6 +82,7 @@
         menu.append(new SeparatorMenuItem());
         menu.append(actions.getAction(ActionId.ABOUT).createMenuItem());
         menu.append(actions.getAction(ActionId.EXIT).createMenuItem());
+        menu.showAll();
     }
 
     @Override
@@ -91,8 +92,7 @@
 
     @Override
     public void onPopupMenu(StatusIcon source, int button, int activateTime) {
-        menu.popup();
-        menu.showAll();
+        menu.popup(source);
     }
 
     /**