Camunda-modeler: Enable connection tool for selected text annotations

[volkergersabeck]

__Description of the Issue__

Currently, text annotations cannot be connected to other elements starting with the text annotation itself. It can be achieved by first selection the target element and creating a new connection from there.

We should offer the connection tool (keyboard shortcut c) also when a text annotation (in BPMN and DMN) is selected.

_As seen in the screencapture, the global connect tool allows to connect gateway :arrow_right: text annotation but not text annotation :arrow_right: gateway._

__Expected Behavior__

The global connect tool allows starting a connection from a text annotation.

[tkhadir]

hello @volkergersabeck, @MaxTru this issue is tagged good first issue i will be honored to contribute if it's possible 😄

[MaxTru]

Dear @tkhadir ,

Sure - we would be happy to retrieve contributions for this :-)

Regards

[tkhadir]

thank you very much @MaxTru can i have more explanations for this issue with and example please ? to be sure that i have understood the goal of it 😄

[nikku]

@tkhadir I've noticed the issue is rather hard to understand without further context.

I've added some more details and a screencast. Please get back to us, if you feel there is still things missing. Please also approach us if you need guidance how to approach this issue.

[tkhadir]

hello @nikku thank you very much for these informations it is more clear with 😄

[tkhadir]

hello @nikku i'm back after analyzing how to enable the connection tool for text annotations, and then i found that in bpmn-js library we need to add 'bpmn:TextAnnotation' to the canStartConnection method to enable it.

thanks in advance for your remarks 😄

[tkhadir]

related MR to enable the connection tool on text annotation ⬆️
https://github.com/bpmn-io/bpmn-js/pull/1428

[MaxTru]

Fixed upstream by https://github.com/bpmn-io/bpmn-js/pull/1428

[pinussilvestrus]

Closed via https://github.com/camunda/camunda-modeler/issues/2194

[MaxTru]

This is not yet closed, since we only implemented the experience for BPMN but not for DMN

[tkhadir]

hello @MaxTru here is my modification related to enabling connect tool for dmn :

https://github.com/bpmn-io/dmn-js/pull/628

("la boucle est bouclée" like we say in french 😄 )

cheers,
Tayeb

[MaxTru]

Fixed upstream by https://github.com/bpmn-io/dmn-js/pull/628

[MaxTru]

Closed via https://github.com/camunda/camunda-modeler/pull/2209