[Commits] r2878 - sandbox/mapgears/geoext.ux/ux/ZoomTo/lib/GeoExt.ux/widgets

[commits at geoext.org]

Author: adube
Date: 2012-02-08 20:32:24 +0100 (Wed, 08 Feb 2012)
New Revision: 2878

Modified:
   sandbox/mapgears/geoext.ux/ux/ZoomTo/lib/GeoExt.ux/widgets/ZoomTo.js
Log:
ZoomTo, window rendering bug fix

Modified: sandbox/mapgears/geoext.ux/ux/ZoomTo/lib/GeoExt.ux/widgets/ZoomTo.js
===================================================================
--- sandbox/mapgears/geoext.ux/ux/ZoomTo/lib/GeoExt.ux/widgets/ZoomTo.js	2012-02-08 14:23:18 UTC (rev 2877)
+++ sandbox/mapgears/geoext.ux/ux/ZoomTo/lib/GeoExt.ux/widgets/ZoomTo.js	2012-02-08 19:32:24 UTC (rev 2878)
@@ -277,11 +277,6 @@
             items: [this.form],
             buttons: windowButtons
         });
-        // force window to render.  forceLayout: true didn't do the trick to
-        // avoid error while dragging the marker without showing the window
-        // first
-        this.window.show();
-        this.window.hide();
     },
 
     /** private: method[onProjctionSelect]
@@ -325,6 +320,12 @@
             return false;
         }
 
+        // make sure the window is rendered first because this method is public
+        // and could be called before the action button was clicked
+        if (!this.window.rendered) { 
+            this.window.render(Ext.getBody()); 
+        } 
+
         var transformedCoords = this.getTransformedCoords(
             values.coordx, values.coordy, values.projection
         );