WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 7 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 17 years ago.

Download all attachments as: .zip

Change History (8)

@mdawaffe
17 years ago

#1 @ryan
17 years ago

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

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

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

  • Resolution fixed deleted
  • Status changed from closed to reopened

#4 @mdawaffe
17 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
17 years ago

  • Owner mdawaffe deleted

#6 @mrmist
17 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.
7 years ago

Note: See TracTickets for help on using tickets.