Opened 13 years ago

Closed 12 years ago

#330 closed defect (fixed)

IE7 Javascript error

Reported by: David Earl Owned by: crschmidt
Priority: major Milestone:
Component: mapnik Version:
Keywords: Cc:

Description

IE7 reports "Unspecified error" (how helpful!) at line 131 char 182, which I assume is this line...

if(OpenLayers?.Element.getStyle(element,'position')=='absolute')break;}while(element=element.offsetParent);element=forElement;do{valueT-=element.scrollTop
0;valueL-=element.scrollLeft0;}while(element=element.parentNode);return[valueL,valueT];};var Rico=new Object();Rico.Corner={round:function(e,options){var e=$(e);this._setOptions(options);var color=this.options.color;if(this.options.color=="fromElement")

(I had assumed this was the search box, but actually that does work, after a fashion - see next ticket).

Change History (4)

comment:1 Changed 13 years ago by David Earl

A bit more info. The debugger highlights this bit

if(element.offsetParent==document.body)

in the following as the error.

var scale=resolution*OpenLayers?.INCHES_PER_UNIT[units]*OpenLayers?.DOTS_PER_INCH;return scale;};OpenLayers?.Util.safeStopPropagation=function(evt){if(evt.stopPropagation){evt.stopPropagation();}

evt.cancelBubble=true;};OpenLayers?.Util.pagePosition=function(forElement){var valueT=0,valueL=0;var element=forElement;do{valueT+=element.offsetTopif(OpenLayers?.Element.getStyle(element,'position')=='absolute')break;}while(element=element.offsetParent);element=forElement;do{valueT-=element.scrollTop
0;valueL+=element.offsetLeft0;if(element.offsetParent==document.body)
0;valueL-=element.scrollLeft0;}while(element=element.parentNode);return[valueL,valueT];};var Rico=new Object();Rico.Corner={round:function(e,options){var e=$(e);this._setOptions(options);var color=this.options.color;if(this.options.color=="fromElement")

color=this._background(e);var bgColor=this.options.bgColor;if(this.options.bgColor=="fromParent") bgColor=this._background(e.offsetParent);this._roundCornersImpl(e,color,bgColor);},changeColor:function(theDiv,newColor){theDiv.style.backgroundColor=newColor;var spanElements=theDiv.parentNode.getElementsByTagName("span");for(var currIdx=0;currIdx<spanElements.length;currIdx++){spanElements[currIdx].style.backgroundColor=newColor;}},changeOpacity:function(theDiv,newOpacity){var mozillaOpacity=newOpacity;var ieOpacity='alpha(opacity='+newOpacity*100+')';theDiv.style.opacity=mozillaOpacity;theDiv.style.filter=ieOpacity;var spanElements=theDiv.parentNode.getElementsByTagName("span");for(var currIdx=0;currIdx<spanElements.length;currIdx++){spanElements[currIdx].style.opacity=mozillaOpacity;spanElements[currIdx].style.filter=ieOpacity;}},reRound:function(theDiv,options){var topRico=theDiv.parentNode.childNodes[0];var bottomRico=theDiv.parentNode.childNodes[2];theDiv.parentNode.removeChild(topRico);theDiv.parentNode.removeChild(bottomRico);this.round(theDiv.parentNode,options);},_roundCornersImpl:function(e,color,bgColor){if(this.options.border)

comment:2 Changed 13 years ago by crschmidt@…

Component: adminslippy_map

It looks like this might actually be an issue in OpenLayers?. If it is, it will be fixed before the end of next week. If it's not, I'll try and find the solution next weekend.

comment:3 Changed 12 years ago by Sebastian@…

Priority: criticalmajor

Does this error still occur or has it been fixed. Not having access to IE myself, I woud be happy if somebody could test this. Toning down to major, this is not critical in any case.

comment:4 Changed 12 years ago by Tom Hughes

Resolution: fixed
Status: newclosed

This doesn't seem to be a problem anymore.

Note: See TracTickets for help on using tickets.