UIDragger emits onClick with onDrop

[tibotiber]

A bit of a follow up to #63.

UIDragger emits both onClick and onDrop when uidrop happens. This is because uidrop comes from the space's up event (UI.ts#L449) which emits onClick (UI.ts#L361). This can be managed in user land (that's what I do so far), but there may be an opportunity for a better behaviour. What are your thoughts on this?

If interested, I can have a go at starting a PR.

[williamngan]

I see. That is a problem. If I understand correctly, should the fix looks like this:

• if dragging happened, only emits 'uidrop'
• if no dragging, only emits 'up'

Agreed that would be a better behavior. If you can start a PR, that would be super helpful!

[tibotiber]

Yes, agreed with the expected behaviour. The only issue is that up is emitted after dragging, when there is no dragging uidrop is not emitted.

I'll have a go at this and open a PR. Probably after #65 is merged to avoid conflicts.

[williamngan]

Thanks! I merged #65.