I'll break out some parts like the html parser bit for separate review. Any other bits need other reviewers Cam?

FYI that the draft SVG 2 specs have changed the capitalization from <solidColor> (as was used in Tiny 1.2) to <solidcolor>. Which removes the HTML parser issue, but also removes any chance at compatibility with Tiny 1.2 content (if any exists out in the wild!).

Cameron, what's your take on this feature? To me it seems like it would be better to have a generic mechanism that works in non-SVG too, but I assume it's still in the SVG 2 draft for a reason?

Yeah, I am pretty ambivalent towards this feature. I was more strongly against it in the past but acquiesced. Tav feels that it is a good mechanism for defining reusable colours in an SVG document. My argument was that you can just use CSS Variables for that. CSS Variables is a harder feature for Inkscape to support, though. I don't really want to be the first mover here. Can we wait to see if this gets interest from other vendors first? (Sorry to do that to you Robert.)

Comment on attachment 8625666 [details] [diff] [review] all files this time. The patch itself looks reasonable, at a skim, modulo the comments about the changed casing of the element name.

removed from SVG 2 spec as nobody wants to implement it.