Koska sivun tärkein toiminto on toteutettu JavaSrciptillä kannataisi lisätä sinne <noscript>Sivu ei toimi ilman JavaScript -tukea!</noscript> huomautus, sillä joku pertti perus internetin käyttäjä ei vältämättä tajuu miksi sivu ei toimi. Tosin ks. pertit ei kyllä tajua ottaa JavaScriptiä pois perusasetuksista kun ei tiedä koko asiasta ja siitä että sen poisottamalla säästyy surfatessaan epäluotettvilla sivuilla viruksilta ja muilta kakoilta joiden installeri on syötetty JavaScript tms. koodiin.
Siiten back to index namiskan teksti ei näy joko poupissa kun tausta on musta ja teksti on musta ja itse nappi on liian pieni. Koska sivu on muutenkin toteutettu JavaScriptillä niin kannattaisi tehdä dynaaminen tekstin väri JavaScriptillä joka vaihtuu tausta värin mukaan. Tämän voi toteuttaa myös PHP:lla sillen että jos linkki on viewimage.php?id=1 niin tekstin väri on valkoinen ja viewimage.php?id=2 taas muuttaa sen mustaksi tai sitten voit tehdä vaikka näin (viewimage.php?id=FFFFFF tai viewimage.php?color=FFFFFF) ---> (viewimage.php?id=000000 tai viewimage.php?color=000000) yms.
<?php
if (!isset($_GET))
{ $color = "000000"; }
else
{ $color = $_GET; }
echo "<a href=\"#\" onClick="your javascript here to close window" style=\"color: #".$color.";\">Back to index</a>";
?>
PS: Huomasin lähdekoodista että se on Ajax/DHTML systeemi, muttä molemmissa on tosin kyse JavaScriptistä. Erona on vaan se että ne käytää XML/CSS/HTML-kieliä extrana.
<body> tagista puuttuu OnLoad funktio joka preloadaa kuvat headeriin laitetulla preload funktiolla.
Valmin preload function saat Dreamweaverista. Sellainen tulee ohjelman peruspaketin mukana.
