Bug Tracker

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#193 closed bug (invalid)

The float property for IE

Reported by: Stefan Petre Owned by:
Priority: undecided Milestone:
Component: ajax Version:
Keywords: float IE Cc: ""
Blocked by: Blocking:

Description

setting and getting float on IE is not working

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

Change History (4)

comment:1 Changed 14 years ago by dave.methvin

Priority: majorminor

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";

comment:2 Changed 14 years ago by Stefan Petre

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

comment:3 Changed 14 years ago by brandon

Resolution: fixed
Status: newclosed

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.

comment:4 Changed 14 years ago by joern

Component: ajax
Priority: blocker
Resolution: invalid
Status: reopenedclosed
Type: bug

SPAMPOLICE

Note: See TracTickets for help on using tickets.