Changeset 7000 for trunk/bp-themes/bp-default/_inc/global.js

Timestamp:

05/06/2013 12:09:17 AM (12 years ago)

Author:

boonebgorges

Message:

When activity comments are deleted via AJAX, remove them from the DOM entirely

Previously, deleted comments were merely hidden using jQuery.fadeOut(). As a
result, when the show/hide comment button was toggled - applying $.show() to
all comment items - the comments would appear again, even though they had been
deleted from the server. Removing them from the DOM ensures that this won't
happen.

See #4844

Props merty

File:

• trunk/bp-themes/bp-default/_inc/global.js (modified) (1 diff)

trunk/bp-themes/bp-default/_inc/global.js

@@ -505 +505 @@
                             child_count++;
                     });
-                    comment_li.fadeOut(200);
+                    comment_li.fadeOut(200, function() {
+                        comment_li.remove();
+                    });
 
                     /* Decrease the "Reply (X)" button count */