Tach!
$(".erinnerung").click(function(){ var $erinnerung = $(".erinnerung"); var $insert_erinnerung = $(".insert_erinnerung"); var $fa = $(".fa ");
Nicht so richtig. Du suchst immer noch die Element bei jedem Click erneut aus dem DOM.
Damit das nicht passiert, sollten die Zuweisungen an die Variablen außerhalb der Eventhandlerfunktion stehen.
Auch das Nach-Außen-Verlagern löst das Problem nicht vollständig. Mindestens das .fa wird voraussichtlich noch an anderen Stellen des Dokuments vorkommen. Und dort soll es nicht mitselektiert werden. Besser im Element bleiben und nur dessen Kinder suchen. Es mag für den Augenblick als geeignete Lösung erscheinen, aber das hat für die Zukunft Nebenwirkungen, die einem dann um die Ohren fliegen, meist ohne dass man sich erinnert, dass die Ursche dort zu finden ist.
Das Suchen in dem/den Elementbereichen lässt sich zwar auch irgendwie auf einmalig reduzieren, aber das erfordert, dass man die Referenzen irgendwo im Element ablegt und darauf zugreift. Das wird alles nur unnötig komplex, so dass mir find()s, die sowieso nur einen kleinen Bereich durchsuchen, der bessere Kompromiss ist.
dedlfix.