Comments on: LightWindow IE6 bug – image size (resizeTo)

By: phobos

phobos — Thu, 04 Mar 2010 14:21:21 +0000

“this.imageCount–;” should be “this.imageCount–-;” (two times minus! Seems that your website has eaten it.)

By: phobos

phobos — Thu, 04 Mar 2010 14:18:38 +0000

I wonder whether the code works, because the if and else statement are the same! Moreover, you’ve duplicated the code for nothing! It would be better to write:
var imgLoaded;
if(ie){
imgLoaded = $(‘lightwindow_image_’+i).complete;
}else{
imgLoaded = (!(typeof $(‘lightwindow_image_’+i).naturalWidth != “undefined” && $(‘lightwindow_image_’+i).naturalWidth == 0));
}
if(imgLoaded){
this.checkImage[i].stop();

var imageHeight = $(‘lightwindow_image_’+i).getHeight();
if (imageHeight > this.resizeTo.height) {
this.resizeTo.height = imageHeight;
}
this.resizeTo.width += $(‘lightwindow_image_’+i).getWidth();
this.imageCount–;

$(‘lightwindow_image_’+i).setStyle({height: ‘100%’, width: ‘100%’});

if (this.imageCount == 0) {this._processWindow();}
}

By: IK

IK — Tue, 05 Jan 2010 05:37:06 +0000

Thank you, saved my day.
BTW, the problem is still there with IE 8 as well.

By: Matthew

Matthew — Fri, 04 Sep 2009 21:15:54 +0000

Fantastic. thank you for this fix, I was pulling my hair out (what’s left of it) looking through the script to find this. Also yes your & and > are still not showing up correctly

By: leon

leon — Wed, 15 Jul 2009 23:28:08 +0000

thanks for this – this bug has been following me like a dog.

BTW you still have & and > in your post code

Marc you just need to co a find and replace on & = & and > = > and it will work again

By: Marc

Marc — Mon, 08 Sep 2008 17:42:16 +0000

This opens in a new window full size and not in lightwindow. Is there anything I can do to make it open in a lightwindow?

By: Jeremy

Jeremy — Mon, 08 Sep 2008 14:39:40 +0000

Sorry about that. I’ve corrected them now. That’s something to always watch out for with these WYSIWYG editors. Had to change that stuff in the code view.

By: john

john — Mon, 08 Sep 2008 00:33:04 +0000

be careful for them &s and >s – they got converted in the post