- From: L. David Baron <dbaron@dbaron.org>
- Date: Thu, 6 Dec 2012 18:35:26 -0800
- To: "Tab Atkins Jr." <jackalmage@gmail.com>
- Cc: Arthur Barstow <art.barstow@nokia.com>, "www-style@w3.org" <www-style@w3.org>
On Friday 2012-11-16 08:19 -0800, Tab Atkins Jr. wrote: > On Fri, Nov 16, 2012 at 7:38 AM, Arthur Barstow <art.barstow@nokia.com> wrote: > > * Seems likeeach usage of CSSRuleList (used in section 7.2) and MediaList > > (used in section 7.4) should have a link to their definition. > > Sure. Fixed in http://dvcs.w3.org/hg/csswg/rev/a639091a0368 > > * Perhaps insertRule() should say what happens if index is already used > > (e.g. replace or fail). > > Neither. It *inserts* it at that index. No replacing goes on. Fixed, along with some other issues, in http://dvcs.w3.org/hg/csswg/rev/11b7f2371aa3 > > * Section 7.4 - not clear why there is a definition of conditionText in this > > section. > > Ah, looks like a leftover. I'll fix. No, this is as intended. Note the text in 7.3: # Since what this condition does varies between the derived # interfaces of CSSConditionRule, those derived interfaces may # specify different behavior for this attribute (see, for example, # CSSMediaRule below). In the absence of such rule-specific # behavior, the following rules apply: and this text in 7.4: # (defined on the CSSConditionRule parent rule) > > * Section 7.6 - for the second definition of supports(), there is a mismatch > > between the "declaration" argument and the use of "conditionText" in the > > text below. (Perhaps the simplest thing to do is > > s/declaration/conditionText). > > Yes. Fixed in http://dvcs.w3.org/hg/csswg/rev/4f6f1a7462fc -David -- 𝄞 L. David Baron http://dbaron.org/ 𝄂 𝄢 Mozilla http://www.mozilla.org/ 𝄂
Received on Friday, 7 December 2012 02:35:56 UTC