Re: Canvas miscellaneous comments

On Tue, 24 Jun 2008, Philip Taylor wrote:
> 
> isPointInPath needs to specify that "inside the current path" is based 
> on the non-zero winding number rule, else it's unclear what "inside" 
> means.

Done.


> Patterns shouldn't be optional: the optionality is harmful to authors, 
> since they can't depend on pattern support; it's harmful for 
> implementors, because they will be incompatible with lots of content if 
> they don't implement patterns; and it doesn't help implementors, since 
> it's not really harder than drawImage and all current implementors chose 
> to support it already.

Done.


> The spec says for getContext: "Arguments other than the contextId must 
> be ignored, and must not cause the user agent to raise an exception (as 
> would normally occur if a method was called with the wrong number of 
> arguments)". That says "must" twice, but it's only one testable 
> assertion, so it would be a tiny bit nicer to say "... must be ignored, 
> and not cause ..."

Simplified that by removing the second half altogether.


> rotate() says "If the angle argument is infinite, the method call must 
> be ignored". That is the default behaviour for canvas methods, so the 
> explicit sentence here should be removed.

Fixed.


> "The stroke() method must [...] fill the combined stroke area using the 
> strokeStyle, attribute." - remove comma after "strokeStyle".

Fixed.


> putImageData: "If any of the arguments to the method are infinite or 
> NaN, the method must raise an NOT_SUPPORTED_ERR exception." - s/an/a/

Fixed.


> "If the anticlockwise argument is false ... or, if the anticlockwise 
> argument is true ..." - that says "<em>true</em>" but plain non-italic 
> "false".

The word "true" is intentionally emphasised in that sentence. Read the 
sentence with stress on the "true" to read it the intended way.

-- 
Ian Hickson               U+1047E                )\._.,--....,'``.    fL
http://ln.hixie.ch/       U+263A                /,   _.. \   _\  ;`._ ,.
Things that are impossible just take longer.   `._.-(,_..'--(,_..'`-.;.'

Received on Tuesday, 5 August 2008 12:11:25 UTC