Eine Lösung?

[dedlfix]

Tach!

ich glaube ich habe es, was sagt ihr dazu?

Wenn du mehrfach auf dasselbe Element zugreifen möchtest, erstell eine Variable, dann muss es nicht mehrfach im DOM gesucht werden. Für jQuery gibts eine Konvention, dass ein solcher Element-Wrapper (Verweis auf ein HTML-Element oder eine Gruppe davon) mit $ anfängt. (Kann man befolgen, muss man aber nicht zwingend.)

var $erinnerung = $(".erinnerung");

Und dann $erinnerung statt $(".erinnerung") verwenden.

isVisible ist gut benannt, ohne $ vorn, weil das nur ein boolescher Wert ist und kein jQuery-Wrapper. Allerdings brauchst du diesen Wert nur ein einziges Mal, so dass eine Variable überflüssig ist. Der Vergleich kann direkt im if notiert werden.

Innerhalb des if und else hast du jeweils 2 Zeilen, die exakt denselben Inhalt haben. Die werden also in jedem Fall auf gleiche Weise ausgeführt und können somit auch außerhalb notiert sein. Wenn du hingegen darauf Wert legst, dass sie in jedem Zweig nicht einfach nur irgendwie umschalten sondern wirklich in die vorgesehene Richtung, egal was vorher für ein Zustand war, dann solltest du die toggle-Zeilen durch Ein- beziehungsweise Ausschalter ersetzen.

Das Wechseln des Textes kannst du nun auch in das if-else verlagern (und entsprechend umschreiben), denn der ist ja auch vom auf/zu-Zustand abhängig.

Und dann wolltest du ja noch bei einer der beiden Bedingungen den Wert vom Input auf Leerstring setzen. Das sollte vielleicht auch noch hinzugefügt werden.

Außerdem war da noch Gunnars Einwand, dass der Text nicht ins Javascript soll. In dem Fall solltest du dir nochmal seine Antwort raussuchen und den Umschalter in zwei Ausführungen erstellen, für jeden Zustand einen, und dann schaltest du nur noch einfach in den anderen Zustand, ohne jedes Element einzeln umzuschalten.

dedlfix.