Doubleclick event (documented) can never work

[xmo]

• Create trivial widget, hook doubleclick event to some operation (e.g. console.log)
• Open editor
• Double click widget

Expected: see operation execute Observed: nothing happens

Internally (in setupWidget) the widgets plugin hooks into doubleclick to trigger edit (by calling widget.edit()), then cancels the event leading to further event handlers being ignored.

As a result, listening to doubleclick will never have any effect unless removeAllListeners is invoked beforehand, which is obviously a terrible idea.

Possible solutions:

• make the handling of doubleclick customisable somehow
• remove doubleclick from widget's documentation and document edit as firing on double click
• ???

[Piotrek Koszuliński]

It's always possible to do something with event, even if it's cancelled. You just need to add a listener with higher priority. For example:

widget.on( 'doubleclick', function() {
    console.log( 'doubleclick' );
}, null, null, 0 );

The standard priority is 10.

However, I don't know why this event is cancelled now. It makes it harder to do something after doubleclick. Manual and automated tests seem to pass without cancelling, so there's a big chance that we'll fix this.

[xmo]

Good point, thanks, I'd missed that. Still, not exactly obvious when you're trying to find out why your even does not fire (and the doc only says that the event is available).

[Piotrek Koszuliński]

I couldn't find any reason why it was cancelled, so I fixed it with ​git:c017cf5.

[Piotrek Koszuliński]

Unfortunately there was a reason why this event was cancelled and why was cancelling editor#doubleclick on widget.wrapper. See #12140. We will partially revert ​git:c017cf5.