Side navigation
#9548 closed bug (fixed)
Opened June 08, 2011 05:40PM UTC
Closed June 14, 2011 08:01PM UTC
Last modified March 09, 2012 05:54AM UTC
animate does not work with fill-opacity css property for svg elements
Reported by: | jquery@silasdavis.net | Owned by: | rwaldron |
---|---|---|---|
Priority: | low | Milestone: | 1.7 |
Component: | effects | Version: | 1.6.1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
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.
Attachments (0)
Change History (6)
Changed June 08, 2011 05:41PM UTC by comment:1
Changed June 08, 2011 05:43PM UTC by comment:2
component: | unfiled → effects |
---|---|
milestone: | 1.next → 1.7 |
priority: | undecided → low |
status: | new → open |
Changed June 08, 2011 05:45PM UTC by comment:3
owner: | → rwaldron |
---|---|
status: | open → assigned |
Dammit dmethvin, you blitzed me!
Changed June 08, 2011 06:09PM UTC by comment:4
Changed June 09, 2011 12:59AM UTC by comment:5
World's fastest bugfix, right here. :)
Sorry I should have added the cssNumber property is on line 6259 of jquery-1.6.1.js