Author: bugman Date: Thu Jun 30 14:13:33 2011 New Revision: 13342 URL: http://svn.gna.org/viewcvs/relax?rev=13342&view=rev Log: The menu binding only occurs if the parent object is supplied to build_menu_item(). Modified: branches/gui_testing/gui/components/spin_view.py branches/gui_testing/gui/menu.py Modified: branches/gui_testing/gui/components/spin_view.py URL: http://svn.gna.org/viewcvs/relax/branches/gui_testing/gui/components/spin_view.py?rev=13342&r1=13341&r2=13342&view=diff ============================================================================== --- branches/gui_testing/gui/components/spin_view.py (original) +++ branches/gui_testing/gui/components/spin_view.py Thu Jun 30 14:13:33 2011 @@ -878,9 +878,9 @@ # The molecule menu entry. menu = wx.Menu() - menu.AppendItem(self.gui.menu.build_menu_item(menu, text="©", icon=paths.icon_16x16.copy, fn=self.gui.user_functions.molecule.copy)) - menu.AppendItem(self.gui.menu.build_menu_item(menu, text="crea&te", icon=paths.icon_16x16.add, fn=self.gui.user_functions.molecule.create)) - menu.AppendItem(self.gui.menu.build_menu_item(menu, text="&delete", icon=paths.icon_16x16.remove, fn=self.gui.user_functions.molecule.delete)) + menu.AppendItem(self.gui.menu.build_menu_item(menu, parent=self, id=1, text="©", icon=paths.icon_16x16.copy, fn=self.gui.user_functions.molecule.copy)) + menu.AppendItem(self.gui.menu.build_menu_item(menu, parent=self, text="crea&te", icon=paths.icon_16x16.add, fn=self.gui.user_functions.molecule.create)) + menu.AppendItem(self.gui.menu.build_menu_item(menu, parent=self, text="&delete", icon=paths.icon_16x16.remove, fn=self.gui.user_functions.molecule.delete)) self.menubar.Append(menu, "&molecule") Modified: branches/gui_testing/gui/menu.py URL: http://svn.gna.org/viewcvs/relax/branches/gui_testing/gui/menu.py?rev=13342&r1=13341&r2=13342&view=diff ============================================================================== --- branches/gui_testing/gui/menu.py (original) +++ branches/gui_testing/gui/menu.py Thu Jun 30 14:13:33 2011 @@ -120,7 +120,7 @@ self.gui.Bind(wx.EVT_MENU_OPEN, self.update_menus) - def build_menu_item(self, menu, id=None, text='', tooltip='', icon=None, fn=None): + def build_menu_item(self, menu, parent=None, id=None, text='', tooltip='', icon=None, fn=None): """Construct and return the menu sub-item. @param menu: The menu object to place this entry in. @@ -151,8 +151,8 @@ element.SetBitmap(wx.Bitmap(icon)) # Bind the menu entry. - if fn: - self.gui.Bind(wx.EVT_MENU, fn, id=id) + if fn and parent: + parent.Bind(wx.EVT_MENU, fn, id=id) # Return the element. return element