Changeset 2d647cf in sasview for src/sas/sasgui/guiframe

Timestamp:

Sep 15, 2016 8:33:57 AM (8 years ago)

Author:

Piotr Rozyczko <rozyczko@…>

Branches:

ESS_GUI, ESS_GUI_Docs, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_iss879, ESS_GUI_iss959, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc

Children:

762ee3e

Parents:

45cab3c

git-author:

Lewis O'Driscoll <lewis.o'driscoll@…> (08/11/16 11:11:15)

git-committer:

Piotr Rozyczko <rozyczko@…> (09/15/16 08:33:57)

Message:

Don't reuse WX IDs in Plotter2D sub-menu (fixes #617)

File:

• src/sas/sasgui/guiframe/local_perspectives/plotting/Plotter2D.py (modified) (2 diffs)

src/sas/sasgui/guiframe/local_perspectives/plotting/Plotter2D.py

@@ -293 +293 @@
 
         wx_id = ids.next()
-        slicerpop.Append(wx_id, '&Save Image')
+        slicerpop.Append(wx_id, '&Save Image', 'Save image as png')
         wx.EVT_MENU(self, wx_id, self.onSaveImage)
 
@@ -320 +320 @@
             if (not item_list == None) and (not len(item_list) == 0) and\
                 self.data2D.name.split(" ")[0] != 'Residuals':
-                # The line above; Not for trunk
-                # Note: reusing menu ids for the sub-menus.  See Plotter1D.
-                for item, wx_id in zip(item_list, self._menu_ids):
+                for item, wx_id in zip(item_list, [ids.next() for i in range(len(item_list))]):
                     try:
                         slicerpop.Append(wx_id, item[0], item[1])