[OpenLayers-Commits] r4242 - trunk/openlayers/lib/OpenLayers/Control

[commits at openlayers.org]

Author: crschmidt
Date: 2007-09-12 10:41:34 -0400 (Wed, 12 Sep 2007)
New Revision: 4242

Modified:
   trunk/openlayers/lib/OpenLayers/Control/OverviewMap.js
Log:
fix for overviewmap open/close button doubleclick resulting in map zoom.
confirmed by erik, caught by acceptance tests. (Closes #757)


Modified: trunk/openlayers/lib/OpenLayers/Control/OverviewMap.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Control/OverviewMap.js	2007-09-12 14:25:34 UTC (rev 4241)
+++ trunk/openlayers/lib/OpenLayers/Control/OverviewMap.js	2007-09-12 14:41:34 UTC (rev 4242)
@@ -221,11 +221,6 @@
             OpenLayers.Event.observe(this.maximizeDiv, 
                           'click', 
                           this.maximizeControl.bindAsEventListener(this));
-            OpenLayers.Event.observe(this.maximizeDiv,
-                          'dblclick',
-                          function(e) {
-                              OpenLayers.Event.stop(e);
-                          });
             this.div.appendChild(this.maximizeDiv);
     
             // minimize button div
@@ -241,13 +236,21 @@
             OpenLayers.Event.observe(this.minimizeDiv, 
                           'click', 
                           this.minimizeControl.bindAsEventListener(this));
-            OpenLayers.Event.observe(this.minimizeDiv,
-                          'dblclick',
-                          function(e) {
-                              OpenLayers.Event.stop(e);
-                          });
             this.div.appendChild(this.minimizeDiv);
             
+            var eventsToStop = ['dblclick','mousedown'];
+            
+            for (var i = 0; i < eventsToStop.length; i++) {
+
+                OpenLayers.Event.observe(this.maximizeDiv, 
+                                         eventsToStop[i], 
+                                         OpenLayers.Event.stop);
+
+                OpenLayers.Event.observe(this.minimizeDiv,
+                                         eventsToStop[i], 
+                                         OpenLayers.Event.stop);
+            }
+            
             this.minimizeControl();
         } else {
             // show the overview map