WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#25575 closed enhancement (fixed)

Lock the compatibility mode in IE

Reported by: azaozz Owned by: azaozz
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: Cc:
PR Number:

Description

IE has several compatibility modes and several ways to control which mode is used. The admin is designed to support the "standards" mode for all IE versions.

We've been using a standards compliant <!DOCTYPE> to set that. However this may fail if there is any output before that tag, or if the user sets the mode manually with the button in the address bar in IE8.

In these cases some admin screens may not render correctly or even be quite unusable. To fix that we can add <meta http-equiv="X-UA-Compatible" content="IE=edge" /> that tells all IE browsers to run in the highest supported document mode of the browser.

Attachments (3)

25575.patch (939 bytes) - added by azaozz 6 years ago.
25575.2.patch (1.0 KB) - added by azaozz 6 years ago.
25575.3.patch (973 bytes) - added by azaozz 6 years ago.

Download all attachments as: .zip

Change History (8)

@azaozz
6 years ago

#1 @azaozz
6 years ago

In 25575.patch: set X-UA-Compatible for IE. This doesn't make any difference in other browsers but shouldn't be set for Chrome Frame as the meta tag overwrites the header send from the server.

It also disables/removes the "Compatibility View" button in the address bar in IE8 and 9, prevents the users from accidentally switching to quirks mode.

Last edited 6 years ago by azaozz (previous) (diff)

#2 @azaozz
6 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 26029:

Add X-UA-Compatible meta tag in the admin, locking the compatibility mode in IE to the highest supported document mode. Fixes #25575.

@azaozz
6 years ago

@azaozz
6 years ago

#3 @azaozz
6 years ago

When there are conditional comments, the meta X-UA-Compatible tag doesn't work (bug in IE8). Best to set that as HTTP header as in 25575.3.patch. Alternatively can remove the conditional tags and add the ie8 class with some JS.

#4 @azaozz
6 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#5 @azaozz
6 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 26089:

Fix the X-UA-Compatible header for IE8, fixes #25575

Note: See TracTickets for help on using tickets.