WordPress.org

Make WordPress Core

Changeset 45869


Ignore:
Timestamp:
08/21/2019 12:26:45 AM (4 months ago)
Author:
SergeyBiryukov
Message:

Customizer: Trim whitespace from custom link URLs.

This complements a similar fix for the Menus screen in [45655].

Props donmhico, audrasjb.
Fixes #47888. See #47723.

Location:
trunk/src/js/_enqueues
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/js/_enqueues/lib/nav-menu.js

    r44680 r45869  
    945945
    946946        addCustomLink : function( processMethod ) {
    947             var url = $('#custom-menu-item-url').val(),
     947            var url = $('#custom-menu-item-url').val().trim(),
    948948                label = $('#custom-menu-item-name').val();
    949949
  • trunk/src/js/_enqueues/wp/customize/nav-menus.js

    r44539 r45869  
    545545                itemName = $( '#custom-menu-item-name' ),
    546546                itemUrl = $( '#custom-menu-item-url' ),
     547                url = itemUrl.val().trim(),
    547548                urlRegex;
    548549
     
    568569                itemName.addClass( 'invalid' );
    569570                return;
    570             } else if ( ! urlRegex.test( itemUrl.val() ) ) {
     571            } else if ( ! urlRegex.test( url ) ) {
    571572                itemUrl.addClass( 'invalid' );
    572573                return;
     
    575576            menuItem = {
    576577                'title': itemName.val(),
    577                 'url': itemUrl.val(),
     578                'url': url,
    578579                'type': 'custom',
    579580                'type_label': api.Menus.data.l10n.custom_label,
Note: See TracChangeset for help on using the changeset viewer.