Closed
Bug 756988
Opened 12 years ago
Closed 12 years ago
Mark more SVG feature strings as supported
Categories
(Core :: SVG, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: longsonr, Assigned: longsonr)
References
Details
Attachments
(1 file)
5.55 KB,
patch
|
jwatt
:
review+
birtles
:
feedback+
|
Details | Diff | Splinter Review |
We do all the animation dom interfaces now (http://www.w3.org/TR/SVG/animate.html#DOMInterfaces) apart from animateColor which is deprecated. I think it's time to mark animation as done too. The only other thing we don't do is wallclock timing I think which we're not planning to do (see http://brian.sol1.net/svg/2010/10/13/its-about-time/)
Assignee | ||
Updated•12 years ago
|
Attachment #625582 -
Attachment is patch: true
Attachment #625582 -
Flags: review?(jwatt)
Attachment #625582 -
Flags: feedback?(birtles)
Comment 1•12 years ago
|
||
Comment on attachment 625582 [details] [diff] [review] patch Seems good to me, assuming Brian agrees.
Attachment #625582 -
Flags: review?(jwatt) → review+
Comment 2•12 years ago
|
||
Comment on attachment 625582 [details] [diff] [review] patch I'm a little hesitant since we don't support wallclock timing, but I think it's ok since we've argued for it to be deprecated (and it probably will be in SVG 2). The description for 'Animation' is that UA supports "the ‘animate’, ‘set’, ‘animateMotion’, ‘animateTransform’, ‘animateColor’ and ‘mpath’ elements" and I'm comfortable saying we do that given that animateColor has already been deprecated.
Attachment #625582 -
Flags: feedback?(birtles) → feedback+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/61e64b3dba61
Flags: in-testsuite+
Target Milestone: --- → mozilla15
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/61e64b3dba61
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Assignee: nobody → longsonr
You need to log in
before you can comment on or make changes to this bug.
Description
•