NO-ISSUE: New BPMN Editor add new icons and improve morphing panels

[kbowers-ibm]

This PR adds new icons to the gateway and task nodes. And beautifies the morphing panels for all node types.







Properties panel examples:

[jomarko]

Is there a way we can help with this? Do some manual review using some live online version of new icons? Or do you have a vsix that we could install locally ?

[kbowers-ibm]

Hi @jomarko,
Thanks for your interest! Tiago showed me how to create an online version, so if you'd like to check it out that'd be great! :)
https://kbowers-ibm.github.io/?path=/story/dev-web-app--web-app

[jomarko]

@kbowers-ibm hi, the icons are nice. Congrats on this PR.

I share some findings, probably some of them not related to this PR, but better to mention them I think, so you can decide if and how to handle them.

01

For the task morphing, there is duplicated number 5. See the attached screenshot.

02

Subprocess morphing seems to work only using mouse and click. Pressing numbers on the keyboard seems to not work. Used chrome on windows.

03

Shape of the 'Timer' and 'Cancel' event in the morph panel is slightly different from icon in the actual node. Do not have strong opinion about it as I am not UX designer. I understand that smaller morph icons do not always allow to use exactly the same shape as node.

[kbowers-ibm]

Hi @jomarko
Thanks for the feedback - appreciate you taking the time!
#1 I think Tiago did that intentionally to easily toggle between a standard task and reusable task, but I agree it's definitely a deviation! Maybe we can follow the behaviour of events where the standard event is 1 and have a standard Task be 1 and a resuable task be something else.
#2 Good point! I didn't touch SubProcesses as part of this PR, but it's probably a good idea to add them in here as opposed to a separate PR.
#3 Good catch! The timer event was intentional as having the 12 ticks was a bit too messy in the smaller morphing panel, but the X could probably be done a bit better!