PDA

View Full Version : Image Thumbnail Viewer II



jscheuer1
04-30-2008, 05:24 PM
1) Script Title: Image Thumbnail Viewer II

2) Script URL (on DD): http://www.dynamicdrive.com/dynamicindex4/thumbnail2.htm

3) Describe problem: In the cleanup:function we have:


this.showcontainer=null

But this.showcontainer hasn't been defined, so this does nothing. If it is intended to get rid of the the one or more:


var showcontainer

variables created in the loadimage:function, it will not. I'm thinking that removing them might not be important, but if it is (perhaps for IE memory leaks), another way must be found. If not, it's not really a bug, just a typo and can safely be removed.

ddadmin
05-01-2008, 03:47 AM
You're right. I think I'll just fix this when I get to updating the script, since as is it doesn't throw any errors either way.