Richtig, dem onclick muss ja auch eine Funktion zugeordnet werden, statt nur eine Anweisung. Versuch es mit

function effekt() {
  new Effect.SlideUp('bildwoche');
}

function aendern() {
  document.getElementById('jan').onclick = effekt;
}

Jetzt bräuchte ich noch ne Möglichkeit das Skript dynamischer zu machen.
Ich habe probiert diese zwei Funktionen in eine zu integrieren aber das hat nicht funktioniert.

Ich hab mir das so vorgestellt:
Die ID´s in den zwei Funktionen sollen über eine Funktion zu bestimmen sein.

Hier nochmal der jetzige Skript (die fett markierten Stellen sollten dann über eine Funktion veränderbar sein:

function aendere() {
document.getElementById('jan').onclick = function () {
new Effect.SlideUp('bildwoche');aendere_zurueck();
}
}
function aendere_zurueck() {
document.getElementById('jan').onclick = function() {
new Effect.SlideDown('bildwoche');aendere('jan');
}
}

Sorry, da unten in der letzten Zeile muss die Funktion mit keinem Parameter aufgerufen werden.

Hallo Computerfreaki,

function aendere() {
 var janID=document.getElementById('jan');
 if (janID.effekt && janID.effekt=="down") {
  janID.onclick = function () {
   new Effect.SlideUp('bildwoche');
  }
  janID.effekt=="up"
 }
 else {
 janID.onclick = function() {
  new Effect.SlideDown('bildwoche');
 }
 janID.effekt="down";
}

also eine einfache IF-Abfrage und entsprechende Zuordnung und den jeweiligen Zustand direkt im (hier übrigens eigentlich überflüssigem) Link speichern.

katy

Kann man jetzt eigentlich auch die ID´s in den Parametern der Funktion auswählen. Weil wenn ich das so mache muss ich für jedes Video oder Bild ein neuen Skript schreiben

Problem hat sich gelöst.
Hier ist die Antwort:
http://github.com/madrobby/scriptaculous/wikis/effect-toggle

und das verstehst du jetzt besser?

Ich farge mich gerade warum ich dir überhaupt noch antworte, in den letzten beiden Threads hast du dann jedesmal selbst eine Lösung präsentiert. Die eine ist Murks und bei dieser hier ist der Bezug zum Problem unklar.

katy