On Mon, 29 Dec 2008 13:33:35 +1100, Kartikaya Gupta
<[email protected]> wrote:
On Mon, 29 Dec 2008 11:30:42 +1100, "Charles McCathieNevile"
<[email protected]> 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
