[Althalos]

Varför fungerar inte det här:
function updateImage() {
if ($(".girl-img").css("background-position-y") == "-96px") $(".girl-img").css("background-position-y", "-474px");
if ($(".girl-img").css("background-position-y") == "-474px") $(".girl-img").css("background-position-y", "-837px");
if ($(".girl-img").css("background-position-y") == "-837px") $(".girl-img").css("background-position-y", "-96px");
window.setTimeout(updateImage(), 2000);
}
updateImage();

tack

[StefanBergfeldt]

Ska det inte vara
window.setTimeout("updateImage()", 2000);

[Althalos]

Det fungerar inte heller. Felmeddelandet i konsollen är att updateImage är undefined. Så det måste finnas något annat fel också.

jag ska även säga att runt alltihop så ligger $(document).ready(function() { });

[dotvoid]

Du ska skicka med en referens till den funktion du ska använda så här:
window.setTimeout(updateImage, 2000);

Alternativt så fungerar det också med en sträng innehållande javascript
window.setTimeout("updateImage()", 2000);

Annars så skickar du ju in returvärdet från updateImage() som första parameter till setTimeout. Det är ju inte riktigt vad du vill.

[dotvoid]

Ursprungligen postat av Althalos

Det fungerar inte heller. Felmeddelandet i konsollen är att updateImage är undefined. Så det måste finnas något annat fel också.

jag ska även säga att runt alltihop så ligger $(document).ready(function() { });

Glömde. Det förklarar ju saken. Läs på lite om scope. Funktionen updateImage() finns ju bara i kontexten av den anonyma funktion du skapar ovan.

Lägg definitionen av updateImage() utanför dokument.ready()

[Althalos]

Ahh... jag började tro att det inte fanns någon logik i det hela, men det finns det naturligtvis alltid. Tack så jättemkt