From 47bcb9bd8df85175e4e271f1e83c5ea47e53a453 Mon Sep 17 00:00:00 2001 From: ahocevar Date: Mon, 2 Mar 2009 15:27:39 +0000 Subject: [PATCH] Possible fix for an issue in IE6 that causes this.element.scrolls to be nullified while being created in getMousePosition. This behavior seems to be due to a bug in IE6 (it appears that two event handlers are running in separate threads, which should never happen). The proposed patch does nothing but improve the coding style of setting scrolls and lefttop, and it seems to fix the issue. p=rdewit, r=me (closes #1952) git-svn-id: http://svn.openlayers.org/trunk/openlayers@8936 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf --- lib/OpenLayers/Events.js | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/lib/OpenLayers/Events.js b/lib/OpenLayers/Events.js index 2515f11d77..22b063e4df 100644 --- a/lib/OpenLayers/Events.js +++ b/lib/OpenLayers/Events.js @@ -774,17 +774,19 @@ OpenLayers.Events = OpenLayers.Class({ } if (!this.element.scrolls) { - this.element.scrolls = []; - this.element.scrolls[0] = (document.documentElement.scrollLeft - || document.body.scrollLeft); - this.element.scrolls[1] = (document.documentElement.scrollTop - || document.body.scrollTop); + this.element.scrolls = [ + (document.documentElement.scrollLeft + || document.body.scrollLeft), + (document.documentElement.scrollTop + || document.body.scrollTop) + ]; } if (!this.element.lefttop) { - this.element.lefttop = []; - this.element.lefttop[0] = (document.documentElement.clientLeft || 0); - this.element.lefttop[1] = (document.documentElement.clientTop || 0); + this.element.lefttop = [ + (document.documentElement.clientLeft || 0), + (document.documentElement.clientTop || 0) + ]; } if (!this.element.offsets) {