Last Comment Bug 512514 - Implement hyperlink targeting of animation elements
: Implement hyperlink targeting of animation elements
Status: RESOLVED FIXED
[parity-Opera][parity-webkit]
: dev-doc-needed
Product: Core
Classification: Components
Component: SVG (show other bugs)
: unspecified
: All All
: -- normal with 1 vote (vote)
: mozilla15
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
http://www.w3.org/Graphics/SVG/Test/2...
Depends on: 492458 512525
Blocks: svg11tests
  Show dependency treegraph
 
Reported: 2009-08-25 11:51 PDT by Jonathan Watt [:jwatt] (Away Jun. 27 - Jul. 13)
Modified: 2012-05-18 06:34 PDT (History)
9 users (show)
longsonr: in‑testsuite+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
wanted


Attachments

Description Jonathan Watt [:jwatt] (Away Jun. 27 - Jul. 13) 2009-08-25 11:51:13 PDT
The SVG spec includes support for starting an animation based on hyperlinking. See:

http://www.w3.org/TR/SVG11/animate.html#IndefiniteValue
http://www.w3.org/TR/smil-animation/#HyperlinkSemantics

Also at least one of the W3C SVG 1.1 Full testsuite tests requires this:

http://www.w3.org/Graphics/SVG/Test/20061213/htmlObjectHarness/full-animate-elem-29-b.html

This bug depends on being able to seek to an earlier point in the timeline: bug 492458.
Comment 1 Jonathan Watt [:jwatt] (Away Jun. 27 - Jul. 13) 2011-03-23 22:35:32 PDT
We basically need to do two things:

 * When we start the timeline after the document is loaded, we need to
   check for a hash and trigger any animation as applicable (be careful
   of pauseAnimations being called before load, that on unpauseAnimations
   things work correctly).

 * We need to have the document listen for 'hashchange' events and trigger
   animations as appropriate. Something like:

  nsCOMPtr<nsIDOMEventTarget> evtTarget = do_QueryInterface(doc);
  evtTarget->AddEventListener(NS_LITERAL_STRING("hashchange"),
                              mEventListener, PR_FALSE);
Comment 2 Robert Longson 2012-04-16 09:48:29 PDT
the patch in bug 512525 fixes this too.
Comment 3 Robert Longson 2012-05-18 06:23:05 PDT
patch in bug 512525 implemented this.

Note You need to log in before you can comment on or make changes to this bug.