Make WordPress Core

Ticket #42321: 42321.7.patch

File 42321.7.patch, 2.7 KB (added by welcher, 6 years ago)

Refreshed patch

  • src/js/_enqueues/lib/nav-menu.js

    diff --git src/js/_enqueues/lib/nav-menu.js src/js/_enqueues/lib/nav-menu.js
    index bed4216595..a85b6f4c5e 100644
     
    835835                                        }
    836836                                }
    837837                        });
     838
     839                        $( '#menu-name' ).on('keyup', _.debounce( function () {
     840                                var menuName = $( document.getElementById( 'menu-name' ) ),
     841                                        menuNameVal = menuName.val();
     842
     843                                if ( ! menuNameVal || menuNameVal === menuName.attr( 'title' ) || ! menuNameVal.replace( /\s+/, '' ) ) {
     844                                        // Add warning for invalid menu name
     845                                        menuName.addClass( 'invalid' );
     846                                } else {
     847                                        // Remove warning for valid menu name
     848                                        menuName.removeClass( 'invalid' );
     849                                }
     850                        }, 500 ));
     851
    838852                        $('#add-custom-links input[type="text"]').keypress(function(e){
    839853                                $('#customlinkdiv').removeClass('form-invalid');
    840854
     
    11661180                        menuName = $('#menu-name'),
    11671181                        menuNameVal = menuName.val();
    11681182                        // Cancel and warn if invalid menu name
    1169                         if( !menuNameVal || menuNameVal == menuName.attr('title') || !menuNameVal.replace(/\s+/, '') ) {
    1170                                 menuName.parent().addClass('form-invalid');
     1183                        if( !menuNameVal || menuNameVal === menuName.attr('title') || !menuNameVal.replace(/\s+/, '') ) {
     1184                                menuName.addClass('invalid');
    11711185                                return false;
    11721186                        }
    11731187                        // Copy menu theme locations
  • src/wp-admin/css/forms.css

    diff --git src/wp-admin/css/forms.css src/wp-admin/css/forms.css
    index 3013434604..82bb5447d1 100644
    textarea[readonly] { 
    238238        color: #72777c;
    239239}
    240240
     241
    241242.form-invalid input,
    242243.form-invalid input:focus,
    243244.form-invalid select,
    244 .form-invalid select:focus {
     245.form-invalid select:focus,
     246input.invalid{
    245247        border-color: #dc3232 !important;
    246248        box-shadow: 0 0 2px rgba( 204, 0, 0, 0.8 );
    247249}
  • src/wp-includes/script-loader.php

    diff --git src/wp-includes/script-loader.php src/wp-includes/script-loader.php
    index e6dc71fcad..292f928276 100644
    function wp_default_scripts( &$scripts ) { 
    17791779                        )
    17801780                );
    17811781
    1782                 // Navigation Menus
    1783                 $scripts->add( 'nav-menu', "/wp-admin/js/nav-menu$suffix.js", array( 'jquery-ui-sortable', 'jquery-ui-draggable', 'jquery-ui-droppable', 'wp-lists', 'postbox', 'json2' ) );
     1782                /**
     1783                 * Navigation Menus
     1784                 *
     1785                 * Adding underscore as a dependency to utilize _.debounce
     1786                 * @since 5.0.0
     1787                 *
     1788                 * @see https://core.trac.wordpress.org/attachment/ticket/42321/
     1789                 */
     1790                $scripts->add( 'nav-menu', "/wp-admin/js/nav-menu$suffix.js", array( 'jquery-ui-sortable', 'jquery-ui-draggable', 'jquery-ui-droppable', 'wp-lists', 'postbox', 'json2', 'underscore' ) );
    17841791                did_action( 'init' ) && $scripts->localize(
    17851792                        'nav-menu',
    17861793                        'navMenuL10n',