Fix: Wrong color used when applying from menu action #4756

[ssidharth010]

Link to issue - #4756
Updated the color map

[CLAassistant]

All committers have signed the CLA.

[jarekdanielak]

Great little change, thank you!

I wonder if we can avoid this problem in the future by having a single source of truth for the colors both in the Desktop Modeler and in the bpmn-js-color-picker?

[nikku]

@jarekdanielak Color picker allows customization.

We could make it possible to retrieve the colors used by the picker (and build the menu from there). That would be my preferred approach. Any other idea you have in mind?

[jarekdanielak]

We could make it possible to retrieve the colors used by the picker (and build the menu from there). That would be my preferred approach. Any other idea you have in mind?

This is exactly what I had in mind.

@ssidharth010 would you like to take your contribution further and draft such a solution or should we take it from here? 😉

[ssidharth010]

@jarekdanielak for sure I can take a look and make the necessary changes.

[jarekdanielak]

Let's merge this anyway to solve the problem.

@ssidharth010 if you want to continue, here's a follow up issue: #4826