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:	major → 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 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:	new → closed

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:	reopened → closed
type:	→ bug

SPAMPOLICE

Side navigation