Re: [svg2] Feedback for latest working draft

*General*
Mentions SVG 1.1 a lot, sometimes erroneously. Should all references be to
SVG 2 in this chapter?

Do we really have to support rendering things by reference that are
display:none. E.g. a pattern or marker that is display:none? No UA
that I know supports display:none if it's written like this...

<div style="display:none"><svg><marker/></svg></div>
<svg><path marker-start="..."></svg>

What's wrong with <defs>?
Specify what should happen when script inserts bad things into an
SVGStringList including

   - null
   - empty string
   - strings containing whitespace (list items are supposed to be separated
   by whitespace, so allowing this breaks round-tripping)
   - strings containing commas (systemLanguage attribute only and this is a
   separator there)
   - probably strings that contain any control characters or other bad
   characters


Chapter 1

delete either 'the' or 'that' towards the end of this sentence...

If the element is within a sub-tree that is instantiated with a
'use<http://www.w3.org/TR/SVG2/struct.html#UseElement>
' element, then the context element is the that
'use<http://www.w3.org/TR/SVG2/struct.html#UseElement>
' element.

feCustom is not part of SVG 1.1 or the latest filter effects specification
and should be removed altogether.
feDropShadow is part of the filter effects specification and not SVG 1.1 so
reference that specification instead.

Chapter 2

Mention *iframe* in embedding by reference perhaps?

Chapter 4
Should SVGAngle support a 'turn' unit to match CSS angles (
https://developer.mozilla.org/en-US/docs/Web/CSS/angle)

currentView
This is a strange thing, it's about the only thing that contains DOMString
elements.
Does any UA implement this?
Why does it have both transform and transformString elements?
It should be reworked or better yet, dropped.

Chapter 5
What happens if you have a conditional processing failure on the parent of
an animated element e.g.

<rect id="r" .../>
<g requiredFeatures="foo">
<set xlink:href="#r" ... />
</g>

No UA I know of supports this. Should we say that conditional processing on
parents of animation elements is ignored?

requiredExtensions
Could we define requiredExtensions for mathml and html support in
foreignObject as being their namespace URLs?

Chapter 6
Add text-shadow as a text property perhaps?
Some text properties e.g. writing-mode are now part of CSS3 aren't they.
Should check and align with that.

Chapter 12
How does non-scaling-stroke affect marker rendering?
How does non-scaling-stroke affect pattern rendering?

Should align image-rendering property with
http://dev.w3.org/csswg/css-images/#the-image-rendering we can't have the
same CSS property
having 2 completely different sets of values.

Chapter18
What does this mean? Is the word 'use' or 'reference' missing?
*SVG 2 will media fragment identifiers*


*Spaces are not allowed in fragment specifications*
So please explain how to write preserveAspectRatio="xMidYMid meet"

Could we drop xpointer syntax?

Chapter19
Add a charset attribute for scripts to match HTML5.
Use case you have a local SVG file which is UTF8 and a local script which
is UTF16
Without mime information or a charset on the script there's no way to get
this to work.

Best regards

Robert