W3C home > Mailing lists > Public > whatwg@whatwg.org > January 2013

Re: [whatwg] proposal: Add support for masking to canvas

From: Rik Cabanier <cabanier@gmail.com>
Date: Tue, 29 Jan 2013 21:55:16 -0800
Message-ID: <CAGN7qDCLdyeRyyA5+7jqCDie8qkwiOBGGQujFE0OhMvu2eh_tg@mail.gmail.com>
To: Boris Zbarsky <bzbarsky@mit.edu>
Cc: WHATWG <whatwg@lists.whatwg.org>, "Tab Atkins Jr." <jackalmage@gmail.com>
On Tue, Jan 29, 2013 at 9:49 PM, Boris Zbarsky <bzbarsky@mit.edu> wrote:

> On 1/30/13 12:37 AM, Rik Cabanier wrote:
>
>>     enum CanvasMaskType { luminosity, alpha};
>>
>
> Last nits.  Need quotes around the enum values.  So:
>
>   enum CanvasMaskType { "luminosity", "alpha" };
>
>      void maskImage(MaskOptions options);
>>
>
> I believe that has to be:
>
>   void maskImage(optional MaskOptions options);
>

Why is it optional?
It's not optional for 'addHitRegion' in the spec either...


>
> to be valid WebIDL.
>
> Apart from that, the IDL parts look fine.  Can't speak to the actual
> graphics bits.  ;)
>
> -Boris
>
>
Received on Wednesday, 30 January 2013 05:55:40 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Wednesday, 30 January 2013 18:48:12 GMT