CSS px set

[john]

in jquery.js there is:

Handle passing in a number to a CSS property if ( value.constructor == Number && type == "curCSS" )

return value + "px";

that +"px" ruins a lot of things and has big impact for users (a lot to change in existing codes)

This ruins opacity in FF $(el).css('opacity', 0.5); will not work I would not send this email if i would find something to fix this in compat plugin.

[john]

Only fontWeight, zIndex, and opacity take non-px numbers.

[joern]

It makes sense to check for the name of the value to set and append px only for those were needed/allowed.

[joern]

Fixed in SVN, revision 1096

[aaron.heimli]

One gotcha here: line-height can accept almost anything:

​http://www.w3schools.com/css/pr_dim_line-height.asp

[joern]

D'oh. Now we are at 5 "fixed" properties...