Changeset 762ee3e in sasview

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:

f0858b6

Parents:

2d647cf

git-author:

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

git-committer:

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

Message:

Update GUI on TEXT_EVT instead of TEXT_EVT_ENTER (closes #186)

File:

• src/sas/sasgui/perspectives/calculator/data_operator.py (modified) (8 diffs)

src/sas/sasgui/perspectives/calculator/data_operator.py

@@ -15 +15 @@
 from sas.sasgui.guiframe.documentation_window import DocumentationWindow
 
-#Control panel width 
+#Control panel width
 if sys.platform.count("win32") > 0:
     PANEL_TOP = 0
@@ -156 +156 @@
 
         wx.EVT_TEXT_ENTER(self.data_namectr, -1, self.on_name)
-        wx.EVT_TEXT_ENTER(self.numberctr, -1, self.on_number)
+        wx.EVT_TEXT(self.numberctr, -1, self.on_number)
         wx.EVT_COMBOBOX(self.data1_cbox, -1, self.on_select_data1)
         wx.EVT_COMBOBOX(self.operator_cbox, -1, self.on_select_operator)
@@ -235 +235 @@
         self.name_sizer.Layout()
 
-    def on_number(self, event=None):
+    def on_number(self, event=None, control=None):
         """
         On selecting Number for Data2
         """
         self.send_warnings('')
-        item = event.GetEventObject()
+        item = control
+        if item is None and event is not None:
+            item = event.GetEventObject()
+        elif item is None:
+            raise ValueError("Event or control must be supplied")
         text = item.GetValue().strip()
         if self.numberctr.IsShown():
@@ -251 +255 @@
                 except:
                     self._set_textctrl_color(self.numberctr, 'pink')
-                    msg = "DataOperation: Number requires a float number."
-                    self.send_warnings(msg, 'error')
-                    return
+                    if event is None:
+                        msg = "DataOperation: Number requires a float number."
+                        self.send_warnings(msg, 'error')
+                    return False
             else:
                 self._set_textctrl_color(self.numberctr, self.color)
@@ -263 +268 @@
         self.draw_output(self.output)
         self.Refresh()
+        return True
 
     def on_select_data1(self, event=None):
@@ -607 +613 @@
             wx.MessageBox(msg, 'Error')
             return
+        if self.numberctr.IsEnabled() and self.numberctr.IsShown():
+            valid_num = self.on_number(control=self.numberctr)
+            if not valid_num:
+                return
         # send data to data manager
         self.output.name = name
@@ -731 +741 @@
         #add plot
         self.graph.add(plot)
-        #draw        
+        #draw
         self.graph.render(self)
 
@@ -985 +995 @@
     window = DataOperatorWindow(parent=None, data=[], title="Data Editor")
     app.MainLoop()
-