Bug Tracker

Ticket #193 (closed bug: invalid)

Opened 2 years ago

Last modified 1 year ago

The float property for IE

Reported by: Stefan Petre Assigned to: anonymous
Type: bug Priority: blocker
Milestone: Component: ajax
Version: Keywords: float IE
Cc: "" Needs:

Description

setting and getting float on IE is not working

The float for IE: get > element.currentStyle.styleFloat set > element.style.styleFloat

Attachments

Change History

Changed 2 years ago by dave.methvin@…

  • priority changed from major to minor

It's in the fix[] map in jQuery().attr(), but as the Firefox name. A fix for .attr() would be to change this

     "float": "cssFloat",

to this:

     "float": jQuery.browser.msie? "styleFloat":"cssFloat",

I think it might be better to completely separate css from attr, there are some cases in there that need to be handled differently. This page has a pretty good rundown of the quirks: http://burstproject.org/tests/testbrowser/style.html

To fix .curCSS(), these lines should work at the beginning:

if ( prop == "float" )
    prop = jQuery.browser.msie? "styleFloat":"cssFloat";

Changed 2 years ago by Stefan Petre

I can't belive this was not included in 1.0.2 .

Changed 2 years ago by brandon

  • status changed from new to closed
  • resolution set to fixed

I would agree that css needs to be seperated from attr and I am working on this. However, in the mean time this bandaid has been applied. In addition to the suggested fix I went ahead and normalized "cssFloat" as well as just "float". Don't use the IE specific "styleFloat" as it will only work for IE.

Fixed in SVN.

Changed 2 years ago by joern

  • priority set to blocker
  • resolution set to invalid
  • status changed from reopened to closed
  • component set to ajax
  • type set to bug

SPAMPOLICE

Note: See TracTickets for help on using tickets.