Changeset 12528

Timestamp:

01/18/2020 12:42:34 PM (5 years ago)

Author:

imath

Message:

xProfile: fix JS Code standards in inline comments

Props passoniate

See #8215

File:

• trunk/src/bp-xprofile/admin/js/admin.js (modified) (10 diffs)

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

@@ -60 +60 @@
     holder.appendChild( newDiv );
 
-    // re-initialize the sortable ui
+    // Re-initialize the sortable ui.
     enableSortableFieldOptions( forWhat );
 
-    // set focus on newly created element
+    // Set focus on newly created element.
     document.getElementById(forWhat + '_option' + theId).focus();
 
@@ -105 +105 @@
 
     document.getElementById( id ).style.display = 'none';
-    // the field id is [fieldtype]option[iterator] and not [fieldtype]div[iterator]
+    // The field id is [fieldtype]option[iterator] and not [fieldtype]div[iterator].
     var field_id = id.replace( 'div', 'option' );
     document.getElementById( field_id ).value = '';
@@ -191 +191 @@
 jQuery( document ).ready( function() {
 
-    // Set focus in Field Title, if we're on the right page
+    // Set focus in Field Title, if we're on the right page.
     jQuery( '#bp-xprofile-add-field #title' ).focus();
 
@@ -200 +200 @@
     } );
 
-    // Set up deleting options ajax
+    // Set up deleting options ajax.
     jQuery( 'a.ajax-option-delete' ).on( 'click', function() {
         var theId = this.id.split( '-' );
@@ -214 +214 @@
     } );
 
-    // Set up the sort order change actions
+    // Set up the sort order change actions.
     jQuery( '[id^="sort_order_"]' ).change(function() {
         if ( jQuery( this ).val() !== 'custom' ) {
@@ -223 +223 @@
     });
 
-    // Show object if JS is enabled
+    // Show object if JS is enabled.
     jQuery( 'ul#field-group-tabs' ).show();
 
-    // Allow reordering of field group tabs
+    // Allow reordering of field group tabs.
     jQuery( 'ul#field-group-tabs' ).sortable( {
         cursor: 'move',
@@ -245 +245 @@
     }).disableSelection();
 
-    // Allow reordering of fields within groups
+    // Allow reordering of fields within groups.
     jQuery( 'fieldset.field-group' ).sortable({
         cursor: 'move',
@@ -264 +264 @@
     })
 
-    // Disallow text selection
+    // Disallow text selection.
     .disableSelection();
 
-    // Allow reordering of field options
+    // Allow reordering of field options.
     enableSortableFieldOptions( jQuery('#fieldtype :selected').val() );
 
-    // Handle title placeholder text the WordPress way
+    // Handle title placeholder text the WordPress way.
     titleHint( 'title' );
 
@@ -314 +314 @@
     } );
 
-    // tabs init with a custom tab template and an "add" callback filling in the content
+    // tabs init with a custom tab template and an "add" callback filling in the content.
     var $tab_items,
         $tabs = jQuery( '#tabs' ).tabs();
@@ -328 +328 @@
             tolerance: 'pointer',
 
-            // When field is dropped on tab
+            // When field is dropped on tab.
             drop: function( ev, ui ) {
                 var $item = jQuery(this), // The tab
                     $list = jQuery( $item.find( 'a' ).attr( 'href' ) ).find( '.connectedSortable' ); // The tab body
 
-                // Remove helper class
+                // Remove helper class.
                 jQuery($item).removeClass( 'drop-candidate' );
 
-                // Hide field, change selected tab, and show new placement
+                // Hide field, change selected tab, and show new placement.
                 ui.draggable.hide( 'slow', function() {
 
-                    // Select new tab as current
+                    // Select new tab as current.
                     $tabs.tabs( 'option', 'active', $tab_items.index( $item ) );
 
-                    // Show new placement
+                    // Show new placement.
                     jQuery(this).appendTo($list).show( 'slow' ).animate( {opacity: '1'}, 500 );
 
-                    // Refresh $list variable
+                    // Refresh $list variable.
                     $list = jQuery( $item.find( 'a' ).attr( 'href' ) ).find( '.connectedSortable' );
                     jQuery($list).find( 'p.nofields' ).hide( 'slow' );
 
-                    // Ajax update field locations and orders
+                    // Ajax update field locations and orders.
                     jQuery.post( ajaxurl, {
                         action: 'xprofile_reorder_fields',