Changeset 33927 for trunk/src/wp-includes/vars.php

Timestamp:

09/05/2015 10:33:01 PM (11 years ago)

Author:

johnbillion

Message:

Introduce a new $is_edge global for the Microsoft Edge browser.

The $is_IE and $is_chrome globals no longer return true for Edge, which is expected as Edge is its own browser and should not be treated like IE. While it might be beneficial for Edge to be treated like Chrome on the client side, it benefits nobody for the $is_chrome global to return true.

See #33193
Props gregrickaby, johnbillion

File:

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

trunk/src/wp-includes/vars.php

@@ -17 +17 @@
 
 global $pagenow,
-        $is_lynx, $is_gecko, $is_winIE, $is_macIE, $is_opera, $is_NS4, $is_safari, $is_chrome, $is_iphone, $is_IE,
+        $is_lynx, $is_gecko, $is_winIE, $is_macIE, $is_opera, $is_NS4, $is_safari, $is_chrome, $is_iphone, $is_IE, $is_edge,
         $is_apache, $is_IIS, $is_iis7, $is_nginx;
 
@@ -49 +49 @@
 
 // Simple browser detection
-$is_lynx = $is_gecko = $is_winIE = $is_macIE = $is_opera = $is_NS4 = $is_safari = $is_chrome = $is_iphone = false;
+$is_lynx = $is_gecko = $is_winIE = $is_macIE = $is_opera = $is_NS4 = $is_safari = $is_chrome = $is_iphone = $is_edge = false;
 
 if ( isset($_SERVER['HTTP_USER_AGENT']) ) {
         if ( strpos($_SERVER['HTTP_USER_AGENT'], 'Lynx') !== false ) {
                 $is_lynx = true;
+        } elseif ( strpos( $_SERVER['HTTP_USER_AGENT'], 'Edge' ) !== false ) {
+                $is_edge = true;
         } elseif ( stripos($_SERVER['HTTP_USER_AGENT'], 'chrome') !== false ) {
                 if ( stripos( $_SERVER['HTTP_USER_AGENT'], 'chromeframe' ) !== false ) {