Closed Bug 721920 Opened 10 years ago Closed 10 years ago
Honor the 'lang' attribute on SVG as we do in HTML
We should support the 'lang' attribute in SVG just as we do in HTML.
Attachment #592325 - Attachment is patch: false
Attachment #592325 - Attachment mime type: text/plain → image/svg+xml
Chrome supports this.
SVG 1.1 uses xml:lang="" for declaring the language of text content, and lang="" just for declaring what language a particular SVG Font glyph is for. I'll file an issue for SVG 2 to use lang="" rather than xml:lang="".
Thanks, that'd be great.
Still need tests, but this works. (The fact that selector matching doesn't use the nsStyleFont struct caught me out for a while, but I figured it out.)
> The fact that selector matching doesn't use the nsStyleFont struct... By which I mean the code for ePseudoClass_lang in nsCSSRuleProcessor.cpp uses GetLang(), not an nsStyleFont...hence the change to nsIContent.h in the patch.
Comment on attachment 598198 [details] [diff] [review] patch r=me
Attachment #598198 - Flags: review?(bzbarsky) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.