Make WordPress Core

Opened 16 years ago

Closed 15 years ago

Last modified 6 years ago

#6327 closed defect (bug) (fixed)

Incorrectly formatted IE conditional comment

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

Description

I believe the format is

<!--[if gte IE 6]>
...
<![endif]-->

Attached.

Attachments (1)

6327.diff (563 bytes) - added by mdawaffe 16 years ago.

Download all attachments as: .zip

Change History (8)

@mdawaffe
16 years ago

#1 @ryan
16 years ago

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

(In [7435]) Fix IE conditional. Props mdawaffe. fixes #6327

#2 @azaozz
16 years ago

Shouldn't that be

<!--[if lte IE 7]>

lower than or equal to IE7? IE8 seems to be doing well for now. And what about IE5/5.5. Most fixes for 6 work there too and somebody may still be using it.

#3 @lloydbudd
16 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#4 @mdawaffe
16 years ago

  • Keywords has-patch 2nd-opinion removed
  • Status changed from reopened to new

This ticket was opened to address a syntactical issue, not a logical one.

That said, I haven't seen anything from IE8 yet, and my understanding is that we officially don't give a hoot about IE5.5.

#5 @mdawaffe
16 years ago

  • Owner mdawaffe deleted

#6 @mrmist
15 years ago

  • Milestone 2.9 deleted
  • Resolution set to fixed
  • Status changed from new to closed

Closing this off as fixed (as it basically was) and wouldn't apply against 2.7.

This ticket was mentioned in Slack in #core-editor by mkaz. View the logs.


6 years ago

Note: See TracTickets for help on using tickets.