Editor throws error when enabling dragging while a tool is active

[sentry-io[bot]]

See comment, wrong issue linked initially

Steps to Reproduce:

• press and hold c in the editor
• the connect tool should now be enabled/disabled rapidly
• click on an element
• The error is shown in the console

Root Cause: This behavior causes a mismatch between Dragging state Active state in the tool manager. When we try to start a connection through clicking, dragging is initiated. Because we cancel the event (https://github.com/bpmn-io/diagram-js/blob/0ec9a1391e9c1f7ace7ad317fe3943a97dabdacc/lib/features/tool-manager/ToolManager.js#L87-L88) while we are initiating the drag operation, it breaks when we access the context immediately after.

Proposed Fix: It is not clear why we cancel dragging in this place. This should be handled either by dragging or a higher component, such as global-connect. This is already the case. No tests break in diagram-js and bpmn-js by just removing this logic, and with my tests, everything still works as expected.

[sentry-io[bot]]

Sentry Issue: BPMN-IO-DEMO-1GX