Results 1 to 2 of 2

Thread: Image Thumbnail Viewer II

  1. #1
    Join Date
    Mar 2005
    Location
    SE PA USA
    Posts
    30,495
    Thanks
    82
    Thanked 3,449 Times in 3,410 Posts
    Blog Entries
    12

    Default Image Thumbnail Viewer II

    1) Script Title: Image Thumbnail Viewer II

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

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

    Code:
    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:

    Code:
    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.
    - John
    ________________________

    Show Additional Thanks: International Rescue Committee - Donate or: The Ocean Conservancy - Donate or: PayPal - Donate

  2. #2
    Join Date
    Aug 2004
    Posts
    10,143
    Thanks
    3
    Thanked 1,008 Times in 993 Posts
    Blog Entries
    16

    Default

    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.

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •