6729 2009-03-24 08:35:16 +0000 Attribute getters/setters should lower case argument on HTML elements in HTML docs 2010-10-04 13:55:16 +0000 1 1 1 Unclassified HTML WG pre-LC1 HTML5 spec (editor: Ian Hickson) unspecified All All VERIFIED FIXED NoReply P2 normal --- 1 hsivonen ian mike public-html-admin public-html-wg-issue-tracking public-html-bugzilla oldest_to_newest 24386 0 hsivonen 2009-03-24 08:35:16 +0000 The spec says that Element.setAttribute() should lower case its argument. This is bad for SVG. Instead, the spec should say that in an HTML document, setAttribute, getAttribute and getAttributeNode on HTMLElement objects must lower case the attribute name argument. See: http://krijnhoetmer.nl/irc-logs/whatwg/20090324#l-236 24444 1 ian 2009-03-31 01:29:58 +0000 The spec already seems to say this... Am I missing something? 24451 2 hsivonen 2009-03-31 11:39:54 +0000 Sorry. I was looking at the interface name and missed the sentence: "This does not apply to XML documents or to elements that are not in the HTML namespace despite being in HTML documents." Indeed, the spec says this for setAttribute. However, it doesn't say it for getAttribute and getAttributeNode. 24454 3 ian 2009-03-31 19:51:24 +0000 Why would getAttribute() or getAttributeNode() do anything special? 24478 4 hsivonen 2009-04-01 15:07:51 +0000 For performance and consistency with past behavior, the actual matching should be exact matching after the argument has been lower cased. http://software.hixie.ch/utilities/js/live-dom-viewer/saved/51 24483 5 ian 2009-04-01 21:16:03 +0000 I agree entirely, but that would argue for mutation when setting, not getting. 24498 6 hsivonen 2009-04-02 07:18:04 +0000 Assume the internal representation is interned "foobar" and assume you call getAttribute("FooBar"). Now you have to lowercase the argument in order to do pointer compares with interned strings. That is, I'm arguing that the spec doesn't match Gecko/WebKit reality when it says the comparison operation should be case-insensitive. 25354 7 ian 2009-05-29 00:43:53 +0000 The spec in fact doesn't say anything about getAttribute() at all. 25355 8 ian 2009-05-29 00:47:35 +0000 Added a section on getAttribute() and getAttributeNS() that says that the argument is lowercased first. 33072 9 mjs 2010-03-14 13:17:06 +0000 This bug predates the HTML Working Group Decision Policy. If you are satisfied with the resolution of this bug, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document: http://dev.w3.org/html5/decision-policy/decision-policy.html This bug is now being moved to VERIFIED. Please respond within two weeks. If this bug is not closed, reopened or escalated within two weeks, it may be marked as NoReply and will no longer be considered a pending comment.