Changeset 12521

Timestamp:

01/16/2020 06:45:34 AM (5 years ago)

Author:

imath

Message:

Admin: fix some JS Coding standards into the Activity screen script

Props passoniate

Fixes #8205

File:

• trunk/src/bp-activity/admin/js/admin.js (modified) (12 diffs)

trunk/src/bp-activity/admin/js/admin.js

@@ -3 +3 @@
 
 /**
- * Activity reply object for the activity index screen
+ * Activity reply object for the activity index screen.
  *
  * @since 1.6.0
@@ -19 +19 @@
         $(document).on( 'click', '#bp-activities-container a.save',   activityReply.send );
 
-        // Close textarea on escape
+        // Close textarea on escape.
         $(document).on( 'keyup', '#bp-activities:visible', function( e ) {
             if ( 27 === e.which ) {
@@ -33 +33 @@
      */
     open : function() {
-        // Hide the container row, and move it to the new location
+        // Hide the container row, and move it to the new location.
         var box = $( '#bp-activities-container' ).hide();
         $( this ).parents( 'tr' ).after( box );
@@ -50 +50 @@
      */
     close : function() {
-        // Hide the container row
+        // Hide the container row.
         $('#bp-activities-container').fadeOut( '200', function () {
 
-            // Empty and unfocus the text area
+            // Empty and unfocus the text area.
             $( '#bp-activities' ).val( '' ).blur();
 
-            // Remove any error message and disable the spinner
+            // Remove any error message and disable the spinner.
             $( '#bp-replysubmit .error' ).html( '' ).hide();
             $( '#bp-replysubmit .waiting' ).hide();
@@ -70 +70 @@
      */
     send : function() {
-        // Hide any existing error message, and show the loading spinner
+        // Hide any existing error message, and show the loading spinner.
         $( '#bp-replysubmit .error' ).hide();
         $( '#bp-replysubmit .waiting' ).show();
 
-        // Grab the nonce
+        // Grab the nonce.
         var reply = {};
         reply['_ajax_nonce-bp-activity-admin-reply'] = $( '#bp-activities-container input[name="_ajax_nonce-bp-activity-admin-reply"]' ).val();
 
-        // Get the rest of the data
+        // Get the rest of the data.
         reply.action    = 'bp-activity-admin-reply';
         reply.content   = $( '#bp-activities' ).val();
@@ -84 +84 @@
         reply.root_id   = $( '#bp-activities-container' ).prev().data( 'root_id' );
 
-        // Make the AJAX call
+        // Make the AJAX call.
         $.ajax({
             data    : reply,
@@ -90 +90 @@
             url     : ajaxurl,
 
-            // Callbacks
+            // Callbacks.
             error   : function( r ) { activityReply.error( r ); },
             success : function( r ) { activityReply.show( r ); }
@@ -99 +99 @@
 
     /**
-     * send() error message handler
+     * send() error message handler.
      *
      * @since 1.6.0
@@ -117 +117 @@
 
     /**
-     * send() success handler
+     * send() success handler.
      *
      * @since 1.6.0
@@ -124 +124 @@
         var bg, id, response;
 
-        // Handle any errors in the response
+        // Handle any errors in the response.
         if ( typeof( xml ) === 'string' ) {
             activityReply.error( { 'responseText': xml } );
@@ -140 +140 @@
         $('#bp-activities-container').fadeOut( '200', function () {
 
-            // Empty and unfocus the text area
+            // Empty and unfocus the text area.
             $( '#bp-activities' ).val( '' ).blur();
 
-            // Remove any error message and disable the spinner
+            // Remove any error message and disable the spinner.
             $( '#bp-replysubmit .error' ).html( '' ).hide();
             $( '#bp-replysubmit .waiting' ).hide();
 
-            // Insert new activity item
+            // Insert new activity item.
             $( '#bp-activities-container' ).before( response.data );
 
-            // Get background colour and animate the flash
+            // Get background colour and animate the flash.
             id = $( '#activity-' + response.id );
             bg = id.closest( '.widefat' ).css( 'backgroundColor' );
@@ -159 +159 @@
 
 $(document).ready( function () {
-    // Create the Activity reply object after domready event
+    // Create the Activity reply object after domready event.
     activityReply.init();
 
-    // On the edit screen, unload the close/open toggle js for the action & content metaboxes
+    // On the edit screen, unload the close/open toggle js for the action & content metaboxes.
     $( '#bp_activity_action h3, #bp_activity_content h3' ).unbind( 'click' );