Re: Some issues with undo manager specification text

2012-09-20 Thread Boris Zbarsky

On 9/20/12 3:25 PM, Ryosuke Niwa wrote:

I guess. The tricky part is that almost all methods and properties on
DOMTransaction are optional. Do you know how one might specify a methods
on a callback interface to be optional?


I'm not sure you can, really.  But you can specify them to be nullable, 
and undefined becomes the null value, so you can at least tell apart 
"null or undefined" and "an actual callable object".


Note that I _think_ attempts to use non-callable objects there should 
throw, in general.  But that's not too obvious to me from the specs.  :(


-Boris




Re: Some issues with undo manager specification text

2012-09-20 Thread Ryosuke Niwa
On Wed, Sep 19, 2012 at 11:00 AM, Boris Zbarsky  wrote:

> I was looking at http://dvcs.w3.org/hg/**undomanager/raw-file/tip/**
> undomanager.htmland
>  ran into some issues:
>
> 1)  No mention of where feedback should be sent.  I have friends in high
> IRC places, so I _think_ I found the right venue, but in general there
> should just be a mailto: link in the spec near the beginning.
>

Right. I originally started the discussion on whatwg but trying to be
chartered by the editing community group. However, I haven't gotten any
formal response from Aryeh with that regard. Nonetheless, the intended
place for feedback is public-webapps.

2)  UndoManager is not written in WebIDL (uses "in" for example).
>

Right, I need to update the definitions using the latest WebIDL draft.

3)  DOMTransaction should presumably be a callback interface.
>
> 4)  The callbacks on DOMTransaction should probably have more restrictive
> signatures than Function.  In particular, Function says nothing about what
> arguments are passed.  At a guess, these should all be something like:
>
>   callback TransactionAction = void ();
>
> instead of Function.


I guess. The tricky part is that almost all methods and properties on
DOMTransaction are optional. Do you know how one might specify a methods on
a callback interface to be optional?

- Ryosuke