Gratulerer i ditt første pull request :-)

I see a couple of issues, though:

• The unit tests fail (check the travis details, run npm test locally next time)
• You have several whitespace-only commits, which you should probably squash into one
• Stopping propagation of the event should be done only if the drag is going to start - in other words, do not stop propagation if I'm holding the shift key or dragging with the right mouse button

Thanks for your guidance @IvanSanchez :-) I'll try to get this right.

Stopping propagation of the event should be done only if the drag is going to start - in other words, do not stop propagation if I'm holding the shift key or dragging with the right mouse button

How would you do this check?

How would you do this check?

It's right there on _onDown, a few lines down. Look closer ;-)

This may affect dragging behavior in some browsers so let's test extensively before merging.

The event now stops to propagate after the three return statements, and I've checked that it still solves the issue.

@turban please squash into one commit.

I have some troubles squashing the commits of this pull requests, so I'm creating a new one.

New pull request with one commit: #4306