Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#25575 closed enhancement (fixed)

Lock the compatibility mode in IE

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: Cc:

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 10 years ago.
25575.2.patch (1.0 KB) - added by azaozz 10 years ago.
25575.3.patch (973 bytes) - added by azaozz 10 years ago.

Download all attachments as: .zip

Change History (8)

@azaozz
10 years ago

#1 @azaozz
10 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, the users won't switch to that accidentally.

Version 0, edited 10 years ago by azaozz (next)

#2 @azaozz
10 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
10 years ago

@azaozz
10 years ago

#3 @azaozz
10 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
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#5 @azaozz
10 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.