sergiotarxz
/
Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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.

git-svn-id: https://develop.svn.wordpress.org/trunk@45869 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov 2019-08-21 00:26:45 +00:00
parent 2fb748bd1c
commit fb189a6d5c
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/js/_enqueues/lib/nav-menu.js
View File

@ -944,7 +944,7 @@
}, },
addCustomLink : function( processMethod ) { addCustomLink : function( processMethod ) {
var url = $('#custom-menu-item-url').val(), var url = $('#custom-menu-item-url').val().trim(),
label = $('#custom-menu-item-name').val(); label = $('#custom-menu-item-name').val();
processMethod = processMethod || api.addMenuItemToBottom; processMethod = processMethod || api.addMenuItemToBottom;

5
src/js/_enqueues/wp/customize/nav-menus.js
View File

@ -544,6 +544,7 @@
var menuItem, var menuItem,
itemName = $( '#custom-menu-item-name' ), itemName = $( '#custom-menu-item-name' ),
itemUrl = $( '#custom-menu-item-url' ), itemUrl = $( '#custom-menu-item-url' ),
url = itemUrl.val().trim(),
urlRegex; urlRegex;
if ( ! this.currentMenuControl ) { if ( ! this.currentMenuControl ) {
@ -567,14 +568,14 @@
if ( '' === itemName.val() ) { if ( '' === itemName.val() ) {
itemName.addClass( 'invalid' ); itemName.addClass( 'invalid' );
return; return;
} else if ( ! urlRegex.test( itemUrl.val() ) ) { } else if ( ! urlRegex.test( url ) ) {
itemUrl.addClass( 'invalid' ); itemUrl.addClass( 'invalid' );
return; return;
} }
menuItem = { menuItem = {
'title': itemName.val(), 'title': itemName.val(),
'url': itemUrl.val(), 'url': url,
'type': 'custom', 'type': 'custom',
'type_label': api.Menus.data.l10n.custom_label, 'type_label': api.Menus.data.l10n.custom_label,
'object': 'custom' 'object': 'custom'