824222 - Implement SVGElement.xml{base|lang|space}

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[David Zbarsky (:dzbarsky)]

Attachment: Patch

I don't know if we actually want these but it's easy.

Comment 1

[David Zbarsky (:dzbarsky)]

Attachment: Implement xmllang and xmlbase

Comment 2

[David Zbarsky (:dzbarsky)]

Attachment: Move xmlspace from SVGStyleElement to SVGElement

Comment 3

[Robert Longson [:longsonr]]

Best ask Cameron as I think this interface is being dropped for SVG2 which is why the original bug 722097 never got landed. You're right it is much easier to implement now, though.

Comment 4

[Robert Longson [:longsonr]]

Comment on attachment 695103 [details] [diff] [review]
Implement xmllang and xmlbase


I thought this was going away in SVG2 Cameron?

Comment 5

[Cameron McCormack (:heycam)]

Comment on attachment 695103 [details] [diff] [review]
Implement xmllang and xmlbase

So we are definitely moving away from using xml:space="" as the mechanism for changing the treatment of white space in <text> elements, but I don't think the working group has discussed the DOM attributes for xml:space, xml:lang and xml:base.  I can bring it up at the next telcon.  (My initial feeling is to avoid implementing them just to avoid encouraging people to rely on them, given they're a bit XML-y.)

Comment 6

[Robert Longson [:longsonr]]

I reckon I'll wait for Cameron to confer with the SVG wg before reviewing these.

Comment 7

[Robert Longson [:longsonr]]

Comment on attachment 695103 [details] [diff] [review]
Implement xmllang and xmlbase

Cancelling review till Cameron is in a position to say what the interface should look like in SVG 2.

Comment 8

[Anne (:annevk)]

xmlBase is equivalent to baseURI so that makes no sense to support. We might even remove xml:base.

Comment 10

[Cameron McCormack (:heycam)]

The SVG WG just resolved to drop these DOM properties: http://www.w3.org/2015/02/11-svg-irc#T00-55-17

Comment 11

[Cameron McCormack (:heycam)]

Let's remove the commented out IDL members though.

Comment 12

[Cameron McCormack (:heycam)]

Attachment: patch

Comment 13

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8563208 [details]
patch

r=me

Comment 14

[Cameron McCormack (:heycam)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9ae6ae84be5a

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/9ae6ae84be5a