Feedback on: Displaying Temporary Pages
Technical:
Too technical
Comments:
For the life of me I could not make this work with my page. I have 27 or so images, many of them part of fading rollovers, and I always got a dialog box stating that some of the scripts may not work, other times the "Please wait ..loading" stayed on the screen, never revealing the contents of the page.It isn't clear where to put the mutiple images part of the code in relationship to the singl image code. What parts of the sinlge image code are necessary for the multiple imge code? It would have been helpfull if you completely wrote out the code for multiple images as opposed to having to add the multiple code parts to the original single image code. Very confusing to newbie web designers.
Worth:
Very worth reading
Length:
Just right
Technical:
Just right
Comments:
I have one small question though. If I want display a gif while the page is loading, I get problems with netscape:
if (document.layers || document.all)
document.write("<img src=myimage.gif>");
The initial myimage.gif shows up, but when the page triggers the other <div> nothing happens. When I view the source the </script> tag and part of the Javascript is gone and the rest of the page is messed up.
Any suggestions would be greatly appreciated.
Henry
Worth:
Very worth reading
Comments:
Why have a variable for each image, if you know that you have N images to load why not maintain a count of images already loaded? This should simplify and generalize the code.
var numberOfImages = 5;
var numberOfImagesLoaded = 0;
function checkImage() {
numberOfImageLoaded++;
if (numberOfImagesLoaded >= numberOfImages) {
...
}
}
Worth:
Worth reading
Length:
Just right
Technical:
Just right
Comments:
Error report regarding http://tech.irt.org/articles/js122
There is an error in the showpage function. An "=" is missing when checking if the client is Internet Explorer.
Thanks for a good site!
// John