Bilderpopup

[misterunknown]

Moin,

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.

Ich habe es jetzt in jeder Funktion einmal. Oder würde das auch funktionieren, wenn ich in der init-Funktion eine globale Variable anlege?

Dann ist mir noch folgendes unklar: in der Funktion showPic() teste ich, ob das Objekt vorhanden ist. Kurz darauf weise ich einer Variablen das Objekt zu:

if(!document.getElementById("piclayer"))mdl.init();  
[...]  
piclayer = document.getElementById('piclayer');

Kann ich das noch irgendwie zusammenfassen?
Folgendes funktioniert nicht:
if(!var piclayer = document.getElementById('piclayer');

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

Das ist gut, mit dem IE hatte ich das noch gar nicht probiert.

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

Das habe ich auch so geändert, sieht viel übersichtlicher aus.

Ich danke dir für deine Tipps.

Grüße Marco