[Webkit-unassigned] [Bug 12073] Implement setCurrentTime() and pauseAnimations() on SVGSVGElement

Mon Nov 21 22:47:49 PST 2011

https://bugs.webkit.org/show_bug.cgi?id=12073

--- Comment #20 from Dirk Schulze <krit at webkit.org>  2011-11-21 22:47:49 PST ---
(In reply to comment #19)
> Instead of patching it, I wonder if we could wipe out the SVG animation support. It is a terrible standard. It is requires lots of complicated code and support infrastructure. A significant amount of work would be need to be done to make it respectable (it modifies dom etc). Other engines do fine without. Hixie removed animations from the Acid3 (the original motivation for implementing them). 
> 
> Are there any real uses for it anywhere?

Yes there are a lot of use cases. And before anyone can remove SVG Animation, we need a reasonable replacement for it. JS is of course one way, but maybe not the best. CSS Animations/Transitions is another way but it is not possible to animate SVG attributes with it (the general use case). The SVG WG is working on finding a replacement, but that no other engine is supporting it is absolutely wrong. Rather there is just one viewer not supporting SVG Animations: IE.

And I also don't understand why it is a terrible standard? You can do a lot more with SVG Animation than with CSS Animations/Transitions, with just one line in the SVG document. It supports the sandwich model and animation of XML attributes as well as CSS properties. It is well defined (at least if you look at SMIL Animations 3.0) with a description of all edge cases.

The work on the DOM changes (animVal/baseVal) is one problem. Neither Niko nor me had enough time to implement it completely.

I had no time to look at the patch yet, but a missing implementation of setCurrentTime() is no reason to remove SVG Animation at this point of time.

In general I do not oppose to push CSS Animations for SVG, but it is still a lot to do before we can think about removing SVG Animation. And even at this time, we will break a lot of content out there.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.