Changeset 40064

Timestamp:

02/16/2017 09:20:45 PM (8 years ago)

Author:

westonruter

Message:

Customize: Skip intercepting non-HTTP(S) links in customizer preview just as jump links are ignored.

Ensures that links with javascript:, mailto: and other protocols work as expected in the customizer preview.

Fixes #39797.

File:

• trunk/src/wp-includes/js/customize-preview.js (modified) (2 diffs)

trunk/src/wp-includes/js/customize-preview.js

@@ -146 +146 @@
             }
 
+            // Allow internal jump links and JS links to behave normally without preventing default.
             isInternalJumpLink = ( '#' === link.attr( 'href' ).substr( 0, 1 ) );
-
-            // Allow internal jump links to behave normally without preventing default.
-            if ( isInternalJumpLink ) {
+            if ( isInternalJumpLink || ! /^https?:$/.test( link.prop( 'protocol' ) ) ) {
                 return;
             }
@@ -335 +334 @@
         }
 
-        // Ignore links with href="#" or href="#id".
-        if ( '#' === $( element ).attr( 'href' ).substr( 0, 1 ) ) {
+        // Ignore links with href="#", href="#id", or non-HTTP protocols (e.g. javascript: and mailto:).
+        if ( '#' === $( element ).attr( 'href' ).substr( 0, 1 ) || ! /^https?:$/.test( element.protocol ) ) {
             return;
         }