From e1fa24df5ca85e2e7f644b87a856decc35959a8c Mon Sep 17 00:00:00 2001 From: crschmidt Date: Wed, 12 Sep 2007 14:41:34 +0000 Subject: [PATCH] fix for overviewmap open/close button doubleclick resulting in map zoom. confirmed by erik, caught by acceptance tests. (Closes #757) git-svn-id: http://svn.openlayers.org/trunk/openlayers@4242 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf --- lib/OpenLayers/Control/OverviewMap.js | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/lib/OpenLayers/Control/OverviewMap.js b/lib/OpenLayers/Control/OverviewMap.js index df33085aa7..582db53b98 100644 --- a/lib/OpenLayers/Control/OverviewMap.js +++ b/lib/OpenLayers/Control/OverviewMap.js @@ -221,11 +221,6 @@ OpenLayers.Control.OverviewMap = OpenLayers.Class(OpenLayers.Control, { 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.Control.OverviewMap = OpenLayers.Class(OpenLayers.Control, { 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