Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upFix document.title setter for SVG documents to match spec changes #8164
Labels
Comments
bors-servo
added a commit
that referenced
this issue
Nov 7, 2015
Prepend, rather than append, SVG title elements. Fixes #8164 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8296) <!-- Reviewable:end -->
bors-servo
added a commit
that referenced
this issue
Nov 7, 2015
Prepend, rather than append, SVG title elements. Fixes #8164 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8296) <!-- Reviewable:end -->
bors-servo
added a commit
that referenced
this issue
Nov 7, 2015
Prepend, rather than append, SVG title elements. Fixes #8164 <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8296) <!-- Reviewable:end -->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See whatwg/html@f7ff53e . The relevant code in Servo is
servo/components/script/dom/document.rs
Line 1495 in dcd207f
Should be easy to fix...
the only potentially tricky bit is that Servo can't parse SVG at the moment, so testing it involves synthesizing an SVG document using DOM calls.Nevermind, a test already exists (or at least it will next time we sync with upstream web-platform-tests),