Bilderpopup

[Kai345]

[latex]Mae  govannen![/latex]

http://misterunknown.de/site/js/piclayer.js

Nachdem es dann irgendwann funktioniert hat, habe ich das Chaos an Funktionen und Befehlen etwas entwirrt und geordnet und unnötiges entfernt. Was sagt ihr dazu? Kann ich einige Sachen besser machen? Was ist schlechter Stil oder lässt sich anders einfacher lösen?

Ein paar Kleinigkeiten fallen mir spontan auf:

Du hast _entschieden_ zu viele document.getElementById("piclayer") im Code. Besser ist es, die Referenz auf dieses Objekt nur _ein_ Mal zu ermitteln und in einer Variablen abzulegen und danach nur noch mit dieser Referenz zu arbeiten.

Dann noch in "keyHandler":

1: if(!evv)evv = window.event;  
2:    if(document.getElementById("piclayer").style.display == "block"){  
3:        evv.preventDefault();  
4:        if(evv.which)taste = evv.which;  
5:        else taste = evv.keyCode;

In Zeile 3  fehlt, falls noch benötigt, die Alternative für alte IE zu evv.preventDefault():

if (evv.preventDefault)  
    evv.preventDefault()  
else  
    evv.returnValue = false;

die Zeilen 4 + 5 sind verwirrend formatiert, "taste" ist eine globale Variable, außerdem würde ich hier das if-else-Konstrukt entfernen :

var taste = evv.which || evv.keyCode;

oder

var taste = (evv.which) ? evv.which : evv.keyCode;

Stur lächeln und winken, Männer!
Kai