- From: Charles McCathieNevile <chaals@opera.com>
- Date: Mon, 29 Dec 2008 17:24:03 +1100
- To: "Kartikaya Gupta" <lists.webapps@stakface.com>
- Cc: public-webapps@w3.org
On Mon, 29 Dec 2008 13:33:35 +1100, Kartikaya Gupta <lists.webapps@stakface.com> wrote: > > On Mon, 29 Dec 2008 11:30:42 +1100, "Charles McCathieNevile" > <chaals@opera.com> wrote: >> >> Please review and send brickbats, comments, etc. ... > 1. s/are not be/are not/ > > 2. Section 2.2, second-last bullet point has an uppercase 'O' in > "Operation" for no apparent reason. > > 3. The description for the "totalArg" parameter of initProgressEvent > says "... the value of this parameter is not a non-negative number ...". > Why not just say "... the value of this parameter is a negative number > ..."? (redundant double negative removal). > > 4. s/prooress/progress/ > > 5. Example 1 in section 3 seems to have been truncated. > > 6. There's a reference to HTML5 in example 3 but it's not hyperlinked to > anything. > > 7. Example 3 has two worky(evt) functions defined. I think one of those > was meant to be whatForYouDoThat(evt). > > 8. Shouldn't the child[1] in example 3 be child[0]? All agreed and fixed. I'll put out another version in a few days that includes all these changes. cheers Chaals -- Charles McCathieNevile Opera Software, Standards Group je parle français -- hablo español -- jeg lærer norsk http://my.opera.com/chaals Try Opera: http://www.opera.com
Received on Monday, 29 December 2008 06:24:51 UTC