Skip to main content

Bug Tracker

Side navigation

#193 closed bug (invalid)

Opened September 09, 2006 06:43PM UTC

Closed October 26, 2006 02:18PM UTC

Last modified June 20, 2007 01:55AM UTC

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

Attachments (0)
Change History (4)

Changed September 11, 2006 06:25PM UTC by dave.methvin comment:1

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

Changed October 10, 2006 04:53AM UTC by Stefan Petre comment:2

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

Changed October 11, 2006 04:24AM UTC by brandon comment:3

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.

Changed October 26, 2006 02:18PM UTC by joern comment:4

component: → ajax
priority: → blocker
resolution: → invalid
status: reopenedclosed
type: → bug

SPAMPOLICE