Changeset 38649 for trunk/src/wp-includes/theme.php

Timestamp:

09/25/2016 04:34:34 AM (10 years ago)

Author:

westonruter

Message:

Customize: Opt to disable IE8 support via conditional comments instead of using unreliable Array.prototype.indexOf feature detection.

Deprecate/remove WP_Customize_Manager::customize_preview_html5() and remove document.head polyfill, both of which were only in place for IE8 support.

Props ryankienstra, westonruter.
Fixes #38021.

File:

• trunk/src/wp-includes/theme.php (modified) (2 diffs)

trunk/src/wp-includes/theme.php

@@ -2152 +2152 @@
  *
  * @since 3.4.0
+ * @since 4.7.0 Support for IE8 and below is explicitly removed via conditional comments.
  */
 function wp_customize_support_script() {
@@ -2159 +2160 @@
 
         ?>
-        <script type="text/javascript">
-                (function() {
-                        var request, b = document.body, c = 'className', cs = 'customize-support', rcs = new RegExp('(^|\\s+)(no-)?'+cs+'(\\s+|$)');
-
-<?php           if ( $cross_domain ): ?>
-                        request = (function(){ var xhr = new XMLHttpRequest(); return ('withCredentials' in xhr); })();
-<?php           else: ?>
-                        request = true;
-<?php           endif; ?>
-
-                        b[c] = b[c].replace( rcs, ' ' );
-                        b[c] += ( window.postMessage && request && Array.prototype.indexOf ? ' ' : ' no-' ) + cs;
-                }());
-        </script>
+        <!--[if lte IE 8]>
+                <script type="text/javascript">
+                        document.body.className = document.body.className.replace( /(^|\s)(no-)?customize-support(?=\s|$)/, '' ) + ' no-customize-support';
+                </script>
+        <![endif]-->
+        <!--[if gte IE 9]><!-->
+                <script type="text/javascript">
+                        (function() {
+                                var request, b = document.body, c = 'className', cs = 'customize-support', rcs = new RegExp('(^|\\s+)(no-)?'+cs+'(\\s+|$)');
+
+                <?php   if ( $cross_domain ) : ?>
+                                request = (function(){ var xhr = new XMLHttpRequest(); return ('withCredentials' in xhr); })();
+                <?php   else : ?>
+                                request = true;
+                <?php   endif; ?>
+
+                                b[c] = b[c].replace( rcs, ' ' );
+                                // The customizer requires postMessage and CORS (if the site is cross domain)
+                                b[c] += ( window.postMessage && request ? ' ' : ' no-' ) + cs;
+                        }());
+                </script>
+        <!--<![endif]-->
         <?php
 }