image.onload funktioniert nicht im ie?

[philomatique]

Hallo,

ganz kurz die Ausgangssituation:

Per Ajax-Updater wird ein Container mit neuem Inhalt gefüllt. Dieser ist eine kleine Galerie. Die Bilder sollen erst angezeigt werden, wenn auch das letzte vollständig geladen ist. Die Ebene wird mittels einem Layer überdeckt, das zum Schluss ausgeblendet wird. Soweit ganz gut. Jetzt habe ich ein JS gebastelt, dass die einzelnen Bilder vorlädt. Das funktioniert super im Firefox und im Safari aufm MAC und einigermaßen im Firefox unter Windows (da er die Ebene ausblendet, bevor alle Bilder vollständig angezeigt - aber dennoch komplett geladen sind). Das große Problem ist jedoch der IE6... Er sagt nämlich, sobald er das zweite Bild erreicht: Fehler: "nicht implementiert". Und ich weiß nicht woran es liegt. Hier ein Codeausschnitt:

myPicture = new Image();
myPicture.src = "path" +"1.jpg";	
myPicture.onload = changePicture(1,myPicture);
				
myPicture1 = new Image();
myPicture1.src = "path" +"2.jpg";	
myPicture1.onload = changePicture(2,myPicture1);

function changePicture(index,picture) {
			
	document.getElementById("picture_"+index).src = picture.src;
			
	++myCounter;
			
	if (myCounter == 2) Effect.Fade('loading', { "duration" : 0.9 } );

}

Habt Ihr eine Idee, woran es liegen könnte? Wenn ich nur ein Bild habe, macht es der IE auch korrekt, nur bei mehreren streikt er.

Danke und Grüße
Phil

 

[Sven Mintel]

Der Fehler liegt in der Eventüberwachung, die Syntax dort ist falsch.

myPicture1.onload = changePicture(2,myPicture1);

...die Funktion changePicture() wird auf diese Weise nicht erst beim onLoad aufgerufen, sondern sofort, wenn diese Skriptpassage vom Browser gelesen wird.

Korrekt sollte es so lauten:

myPicture1.onload = function(){changePicture(2,myPicture1);}

 

[philomatique]

Hi Sven,

danke für Deine Antwort. Das hatte ich auch schon probiert. Allerdings nicht, indem ich den Funktionsaufruf sondern den kompletten Inhalt in die "neue" Funktion gepackt habe. Das werde ich gleich nochmal probieren. Sehr gute Idee.

Danke und viele Grüße
Phil

 

[Sven Mintel]

das mit dem kompletten Inhalt sollte eigentlich auch funktionieren, es ist ja aus JS-Sicht dasselbe, nur anders geschrieben.