Changeset 7412

Timestamp:

10/11/2013 05:50:30 PM (13 years ago)

Author:

boonebgorges

Message:

Use delegation for bulk message actions.

This ensures that items added after AJAX actions are available for AJAX
deletion.

See #5070

Props imath

Location:

trunk

Files:

• bp-templates/bp-legacy/js/buddypress.js (modified) (1 diff)
• bp-themes/bp-default/_inc/global.js (modified) (1 diff)

trunk/bp-templates/bp-legacy/js/buddypress.js

@@ -1219 +1219 @@
         
         /* Bulk delete messages */
-        jq("#delete_inbox_messages, #delete_sentbox_messages").on( 'click', function() {
+        jq( 'body.messages #item-body div.messages' ).on( 'click', '.messages-options-nav a', function() {
+                if( -1 == jq.inArray( this.id ), Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) ) ) {
+                        return;
+                }
+                
                 checkboxes_tosend = '';
                 checkboxes = jq("#message-threads tr td input[type='checkbox']");

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

@@ -1203 +1203 @@
         
         /* Bulk delete messages */
-        jq("a#delete_inbox_messages, a#delete_sentbox_messages").click( function() {
+        jq( 'body.messages #item-body div.messages' ).on( 'click', '.messages-options-nav a', function() {
+                if ( -1 == jq.inArray( this.id ), Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) ) ) {
+                        return;
+                }
+                
                 checkboxes_tosend = '';
                 checkboxes = jq("#message-threads tr td input[type='checkbox']");