Re: [selectors-api] Minor comments on the spec text

2007-06-26 Thread liorean
On 26/06/07, Lachlan Hunt [EMAIL PROTECTED] wrote: Lachlan Hunt wrote: liorean wrote: I think it wouldn't hurt to specify that an exception should be propagated (or rethrown?) out to the caller. Surely, if it were necessary to state that, it should be in DOM Level 3 Core where

[selectors-api] Minor comments on the spec text

2007-06-25 Thread Philip Taylor
Some comments from reading through r1.23 (http://dev.w3.org/cvsweb/~checkout~/2006/webapi/selectors-api/Overview.html?rev=1.23content-type=text/html;%20charset=utf-8): Abstract: [CSS21] isn't italicised. (often simply referred to as selector) - the grammar sounds wrong; should it be ...as a

Re: [selectors-api] Minor comments on the spec text

2007-06-25 Thread Magnus Kristiansen
On Mon, 25 Jun 2007 18:03:40 +0200, Lachlan Hunt [EMAIL PROTECTED] wrote: Philip Taylor wrote: I'd also remove the var rows = null; and put the var inside the loop to simplify the code, though perhaps that's a bad idea because JS scoping is weird. Correct me if I'm wrong, but if it

Re: [selectors-api] Minor comments on the spec text

2007-06-25 Thread liorean
Philip Taylor wrote: I'd also remove the var rows = null; and put the var inside the loop to simplify the code, though perhaps that's a bad idea because JS scoping is weird. On 25/06/07, Lachlan Hunt [EMAIL PROTECTED] wrote: Correct me if I'm wrong, but if it were declared inside the