This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 22051 - DOMTokenList clarifications
Summary: DOMTokenList clarifications
Status: RESOLVED FIXED
Alias: None
Product: WebAppsWG
Classification: Unclassified
Component: DOM (show other bugs)
Version: unspecified
Hardware: All All
: P2 normal
Target Milestone: ---
Assignee: Anne
QA Contact: public-webapps-bugzilla
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-15 19:10 UTC by Ms2ger
Modified: 2013-09-06 15:24 UTC (History)
2 users (show)

See Also:


Attachments

Description Ms2ger 2013-05-15 19:10:14 UTC
<Ms2ger> DOMTokenList
<Ms2ger> I can't find the part where we say that it should be populated from the attribute value
<annevk> open bug iirc
<Ms2ger> Ah
<annevk> not entirely sure anymore what I thought the right solution for that would be at the time
<annevk> it should be pretty clear that setting it would just run the set parser
<Ms2ger> So, my main question is this
<Ms2ger> rel="test test"
<Ms2ger> What is relList.length?
<annevk> 1
<annevk> note that this is a change from DOMTokenList's original design (we made that two months back or so)
<annevk> we changed it into a set for perf and simplicity
<Ms2ger> http://dom.spec.whatwg.org/#concept-set-serializer doesn't seem to be clear about the order
<Ms2ger> Also, "running the set serializer for the [...] list"?
<annevk> sorry, it's an ordered set
<annevk> I guess "add the collected string" should say "append ..."
Comment 1 Anne 2013-05-20 03:18:09 UTC
https://github.com/whatwg/dom/commit/a318480b61924a0d30ab35a35e7e47df6852a2a3

Did we want to fix anything else as part of this bug?
Comment 2 Ms2ger 2013-05-20 07:45:31 UTC
<Ms2ger> I can't find the part where we say that it should be populated from the attribute value
Comment 3 Anne 2013-05-20 07:49:21 UTC
That part sounds like a duplicate of bug 20660.
Comment 4 Anne 2013-09-06 15:24:50 UTC
I think I fixed a bunch of this stuff recently. I'm going to mark this FIXED given the open bug in comment 3 and these commits:

https://github.com/whatwg/dom/commit/52bec56bdbff045a9d0381d967b6c468ba19ff87
https://github.com/whatwg/dom/commit/bab1a08be1f823b72ea681f40f7ecb9c43a44400