Moved configurator init logic into function for GUI/interact() to call

Packages/vcs/Lib/Canvas.py

@@ -468,8 +468,7 @@ def start(self,*args,**kargs):
         self.interact(*args,**kargs)
 
     def interact(self,*args,**kargs):
-      if self.configurator is not None:
-        self.configurator.show()
+      self.configure()
       self.backend.interact(*args,**kargs)
 
     def _datawc_tv(self, tv, arglist):
@@ -941,7 +940,7 @@ def __init__(self, gui = 0, mode = 1, pause_time=0, call_from_gui=0, size=None,
 
         self._animate = self.backend.Animate( self )
 
-        self.configurator = configurator.Configurator(self, show_on_update=(backend != "vtk") )
+        self.configurator = None
 
 ## Initial.attributes is being called in main.c, so it is not needed here!
 ## Actually it is for taylordiagram graphic methods....
@@ -966,12 +965,21 @@ def __init__(self, gui = 0, mode = 1, pause_time=0, call_from_gui=0, size=None,
            else:
               shutil.copy2(os.path.join(*pth),user_init)
 
-	called_initial_attributes_flg = 1
+        called_initial_attributes_flg = 1
         self.canvas_template_editor=None
         self.ratio=0
         self._user_actions_names=['Clear Canvas','Close Canvas','Show arguments passsed to user action']
         self._user_actions = [self.clear, self.close, self.dummy_user_action]
 
+    def configure(self):
+        self.configurator = configurator.Configurator(self)
+        self.configurator.update()
+        self.configurator.show()
+
+    def endconfigure(self):
+        self.configurator.detach()
+        self.configurator = None
+
     def processParameterChange( self, args ):
         self.ParameterChanged( args )
 
@@ -3732,8 +3740,7 @@ def set_convert_labels(copy_mthd,test=0):
         if dn is not None:
           self.display_names.append(result.name)
           if result.g_type in ("3d_scalar", "3d_vector") and self.configurator is not None:
-            self.configurator.detach()
-            self.configurator = None
+            self.endconfigure()
           if self.backend.bg == False and self.configurator is not None:
             self.configurator.update()
 

Packages/vcs/Lib/configurator.py

@@ -143,10 +143,9 @@ def array_strings(template, array):
     return strings
 
 class Configurator(object):
-    def __init__(self, canvas, show_on_update=False):
+    def __init__(self, canvas):
         self.canvas = canvas
         self.backend = canvas.backend
-        self.show_on_update = show_on_update
         self.interactor = None
         self.display_strings = {}
         self.displays = []
@@ -260,10 +259,6 @@ def update(self):
             # This is an attribute used internally; might break
             display._template_origin = new_template.name
 
-        if self.show_on_update:
-            self.show()
-            self.show_on_update = False
-
     def detach(self):
         if self.toolbar is not None:
             self.toolbar.detach()
@@ -363,6 +358,7 @@ def click(self, object, event):
     def show(self):
         if len(self.displays) == 0:
             return
+        self.place()
         self.toolbar.show()
         self.marker_button.show()
         self.text_button.show()