245408 - 'position' shouldn't apply to SVG elements

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Hixie (not reading bugmail)]

The 'position' property shouldn't apply to SVG elements.

TESTCASE: http://www.hixie.ch/tests/adhoc/svg/mixed/004.xml

Comment 1

[Boris Zbarsky [:bzbarsky]]

I assume there's a reason for this (and an explicit mention of it in the SVG spec)?

Comment 2

[Hixie (not reading bugmail)]

Yeah -- 'position' is not one of the properties mentioned in SVG 1.1 section
6.1, so it doesn't apply to SVG.

See: http://www.w3.org/TR/SVG/styling.html

Comment 3

[Alex Fritze]

Attachment: patch to nsCSSFrameConstructor.cpp

How about this?

Comment 4

[Boris Zbarsky [:bzbarsky]]

Oh, I see... so CSS only applies to SVG if it's in that white-list.... fun....

Comment 5

[Boris Zbarsky [:bzbarsky]]

Alex, I'm not going to be in a position to really review anything other than the
things I've already promised people I'd try to get to till July 11 at this
point...  Try dbaron or roc?

Comment 6

[Alex Fritze]

(In reply to comment #4)
> Oh, I see... so CSS only applies to SVG if it's in that white-list.... fun....

Well, "position" doesn't really make sense for most SVG elements - you can apply
it but it hasn't got any effect, apart from maybe inheriting down to other
elements. I'm sure the same is true for many CSS property / XHTML elements
combinations.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 149897 [details] [diff] [review]
patch to nsCSSFrameConstructor.cpp

Please add "-p" to the options you use for diff.

Ian, do you think the |svg| element should be allowed to be absolutely
positioned?  The spec probably says not, but it certainly does make sense when
mixing languages.

Comment 8

[Hixie (not reading bugmail)]

As far as I'm concerned, the <svg> element is a replaced element as far as the
outer CSS world goes. So I'd say sure, position away. Or float, or put inline,
or whatever. We should put that in CSS3 Box...

Comment 9

[Alex Fritze]

David, thanks for the review. Do I need sr as well or is r sufficient, given
that the code is "SVG-only"?

Comment 10

[Alex Fritze]

checked in