r23607 - in /trunk/gui/input_elements: sequence.py sequence_2D.py -- May 30, 2014 - 09:55

Author: bugman
Date: Fri May 30 09:55:43 2014
New Revision: 23607

URL: http://svn.gna.org/viewcvs/relax?rev=23607&view=rev
Log:
Fix for bug #22112 (http://gna.org/bugs/?22112).

This is the GUI failure when setting list values via the sequence windows, 
launched from user
function windows fails on Mac OS X.  The problem was two fold.  First the 
Sequence and Sequence_2D
windows from wx.Dialog should not be terminated via the Destroy() method, as 
wx.Dialog.Destroy()
appears to be horribly broken on Macs.  Additionally, the setting of data in 
the Sequence and
Sequence_2D windows is now performed using the thread-safe wx.CallAfter() 
method.


Modified:
    trunk/gui/input_elements/sequence.py
    trunk/gui/input_elements/sequence_2D.py

Modified: trunk/gui/input_elements/sequence.py
URL: 
http://svn.gna.org/viewcvs/relax/trunk/gui/input_elements/sequence.py?rev=23607&r1=23606&r2=23607&view=diff
==============================================================================
--- trunk/gui/input_elements/sequence.py        (original)
+++ trunk/gui/input_elements/sequence.py        Fri May 30 09:55:43 2014
@@ -741,8 +741,8 @@
         @type event:    wx event
         """
 
-        # Destroy the window.
-        self.Destroy()
+        # Close the window.
+        self.Close()
 
 
     def delete(self, event):

Modified: trunk/gui/input_elements/sequence_2D.py
URL: 
http://svn.gna.org/viewcvs/relax/trunk/gui/input_elements/sequence_2D.py?rev=23607&r1=23606&r2=23607&view=diff
==============================================================================
--- trunk/gui/input_elements/sequence_2D.py     (original)
+++ trunk/gui/input_elements/sequence_2D.py     Fri May 30 09:55:43 2014
@@ -126,7 +126,7 @@
         self.sel_win = Sequence_window_2D(name=self.name, 
seq_type=self.seq_type, value_type=self.value_type, titles=self.titles, 
dim=self.dim)
 
         # Set the values.
-        self.sel_win.SetValue(self.GetValue())
+        wx.CallAfter(self.sel_win.SetValue, self.GetValue())
 
         # Show the window.
         if status.show_gui: