animate does not work with fill-opacity css property for svg elements

[jquery@…]

Code such as the following:

$(vertex).animate({ "fill-opacity": 1.0 }, "slow", "swing", function () { animateVertexQueue(vertexLabels); });

Does not cause the fill-opacity css property be changed as you would expect. Looking through the code the error is an omission from the cssNumber property, it reads:

	cssNumber: {
		"zIndex": true,
		"fontWeight": true,
		"opacity": true,
		"zoom": true,
		"lineHeight": true,
		"widows": true,
		"orphans": true
	},

But should read:

	cssNumber: {
		"zIndex": true,
		"fontWeight": true,
		"opacity": true,
		"fillOpacity": true,
		"zoom": true,
		"lineHeight": true,
		"widows": true,
		"orphans": true
	},

That is with fillOpacity added as a flag. This stops "px" from being appended to the property value in the easing, which cause the property change to fail.

[anonymous]

Sorry I should have added the cssNumber property is on line 6259 of jquery-1.6.1.js

[Rick Waldron]

Dammit dmethvin, you blitzed me!

[Rick Waldron]

​https://github.com/jquery/jquery/pull/409

[dmethvin]

World's fastest bugfix, right here. :)

[rwldrn]

Landing pull request 409. Adds fillOpacity to internal cssNumber. Fixes #9548.

More Details:

• ​https://github.com/jquery/jquery/pull/409
• ​http://bugs.jquery.com/ticket/9548 Changeset: 5eef5917fdb399ace2698154c4cd7bbd24f13182