Could you try again with r1465? Thanks for the excellent bug report, btw.
Cheers, /Per On Mon, Nov 3, 2008 at 6:05 PM, bkc <[EMAIL PROTECTED]> wrote: > > I have a production site that is using this svn url: > http://svn.mochikit.com/mochikit/trunk/packed/MochiKit/MochiKit.js via > an external > > after the last svn update blindUp($(id)) isn't working (see details > below) > > I have checked out a bunch of back revs. It works in rev 1448, fails > in rev 1449 and onwards. > > It fails in IE7 and IE 6.1 > > it does work fine w/ FF 2x > > it looks to me like getElementDimensions() has changed in 2 ways: > > 1. tries to set the style.display to something other than "" > 2. tries to set the style.display for tables to "table" which > apparently doesn't work in IE. > > === > > so in rev 1449 I get the error in MochiKit.js line 4197 > > working the callstack upwards from my code: > > if(new_state == 'closed') { > > blindUp($(id)); > > if(indicator) { > > removeElementClass(indicator, 'hide'); > > addElementClass(indicator, 'show'); > > } > > IE debugger command window: > > id > > "master_headertable" > > typeof $(id) > > "object" > > $(id).id > > "master_headertable" > > up the callstack to blindUp() > > MochiKit.Visual.blindUp=function(_6d7,_6d8){ > var d=MochiKit.DOM; > _6d7=d.getElement(_6d7); > var _6da=MochiKit.Style.getElementDimensions(_6d7,true); /* line 6590 > fails here) > var _6db=d.makeClipping(_6d7); > > it is failing in line 6590 > > command window: > > typeof _6d7 > > "object" > > _6d7.id > > "master_headertable" > > up the callstack to getElementDimensions > > getElementDimensions:function(elem,_448){ > var self=MochiKit.Style; > var dom=MochiKit.DOM; > if(typeof (elem.w)=="number"||typeof (elem.h)=="number"){ > return new self.Dimensions(elem.w||0,elem.h||0); > } > elem=dom.getElement(elem); > if(!elem){ > return undefined; > } > var disp=self.getStyle(elem,"display"); > if(disp=="none"||disp==""||typeof (disp)=="undefined"){ > var s=elem.style; > var _44d=s.visibility; > var _44e=s.position; > var _44f=s.display; > s.visibility="hidden"; > s.position="absolute"; > s.display=self._getDefaultDisplay(elem); /* line 4197 fails here */ > var _450=elem.offsetWidth; > var _451=elem.offsetHeight; > > failing at line 4197 > > command window: > > elem.id > > "master_headertable" > > > self._getDefaultDisplay(elem) > > "table" > > typeof s > > "object" > > > s.display > > "none" > > > s.position > > "absolute" > > /* try setting the display mode */ > s.display = 'block' > > "block" > > > > s.display = 'table' > > Could not get the display property. Invalid argument. > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "MochiKit" group. To post to this group, send email to mochikit@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/mochikit?hl=en -~----------~----~----~----~------~----~------~--~---