Changeset 41311

Timestamp:

08/24/2017 05:51:19 PM (7 years ago)

Author:

wonderboymusic

Message:

Docs: improve JS docs for tags.js

Props atimmer, jipmoors.
Fixes #41069.

File:

• trunk/src/wp-admin/js/tags.js (modified) (5 diffs)

trunk/src/wp-admin/js/tags.js

@@ -1 +1 @@
 /* global ajaxurl, wpAjax, tagsl10n, showNotice, validateForm */
+
+
+
+
+
+
+
 
 jQuery(document).ready(function($) {
 
+
+
+
+
+
+
+
+
+
+
     $( '#the-list' ).on( 'click', '.delete-tag', function() {
         var t = $(this), tr = t.parents('tr'), r = true, data;
+
         if ( 'undefined' != showNotice )
             r = showNotice.warn();
+
         if ( r ) {
             data = t.attr('href').replace(/[^?]*\?/, '').replace(/action=delete/, 'action=delete-tag');
+
+
+
+
+
+
+
+
+
             $.post(ajaxurl, data, function(r){
                 if ( '1' == r ) {
                     $('#ajax-response').empty();
                     tr.fadeOut('normal', function(){ tr.remove(); });
-                    // Remove the term from the parent box and tag cloud
+
+                    /**
+                     * @summary Remove the term from the parent box and the tag cloud
+                     *
+                     * `data.match(/tag_ID=(\d+)/)[1]` matches the term id from the data variable.
+                     * This term id is then used to select the relevant HTML elements:
+                     * The parent box and the tag cloud.
+                     */
                     $('select#parent option[value="' + data.match(/tag_ID=(\d+)/)[1] + '"]').remove();
                     $('a.tag-link-' + data.match(/tag_ID=(\d+)/)[1]).remove();
+
                 } else if ( '-1' == r ) {
                     $('#ajax-response').empty().append('<div class="error"><p>' + tagsl10n.noPerm + '</p></div>');
                     tr.children().css('backgroundColor', '');
+
                 } else {
                     $('#ajax-response').empty().append('<div class="error"><p>' + tagsl10n.broken + '</p></div>');
@@ -24 +61 @@
                 }
             });
+
             tr.children().css('backgroundColor', '#f33');
         }
+
         return false;
     });
 
+
+
+
+
+
+
+
     $( '#edittag' ).on( 'click', '.delete', function( e ) {
         if ( 'undefined' === typeof showNotice ) {
@@ -34 +80 @@
         }
 
+
         var response = showNotice.warn();
         if ( ! response ) {
@@ -40 +87 @@
     });
 
+
+
+
+
+
+
+
+
+
     $('#submit').click(function(){
         var form = $(this).parents('form');
@@ -46 +102 @@
             return false;
 
+
+
+
+
+
+
+
         $.post(ajaxurl, $('#addtag').serialize(), function(r){
             var res, parent, term, indent, i;