Opened 11 years ago
Closed 11 years ago
#28151 closed defect (bug) (fixed)
IE <html> tag uses wrong conditional logic
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.0 | Priority: | normal |
| Severity: | normal | Version: | 3.2 |
| Component: | Bundled Theme | Keywords: | has-patch commit |
| Focuses: | Cc: |
Description
Currently all bundled themes from twentyeleven use an OR operator ( | ) for the last <html> conditional statement:
<!--[if !(IE 7) | !(IE 8) ]><!--> <html <?php language_attributes(); ?>> <!--<![endif]-->
This means that both IE 7 and 8 parse the default <html> tag but apparently they just use the first one they are given (the last if statement is redundant).
Ref: http://msdn.microsoft.com/en-gb/library/ms537512(v=vs.85).aspx#syntax
Attachments (1)
Change History (6)
Note: See
TracTickets for help on using
tickets.
The patch simply replaces | with & (the AND operator) and removes an extra space from eleven, twelve and thirteen. Best way I could think of testing it was to add in some junk text inside each statement (so with the OR operator, IE8 prints the text in the default <html> block)