-
Notifications
You must be signed in to change notification settings - Fork 420
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Align colors with Camunda Modeler #477
Conversation
WIP because Andi needs to review this too. Code is ready for a review otherwise. |
348ef8a
to
b0d967f
Compare
I set the definitions to .djs-container scope rather than root scope. |
478d38d
to
531d4e9
Compare
I modified the context pad hover color that Andi mentionned in his review. The rest is compatible with the v1 version of the designs. So we can merge this IMO. |
531d4e9
to
15f397b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the review we want to remove orange in favor of blue. I've gone ahead and did that as I had done it in my proposal.
I've also refactored the CSS variable names. I don't think that names like --gainsboro
are very helpful. I have no idea what that is let alone that it's supposed to be a color.
* refactor color variable names * remove orange * fix visibility of bendpoint and segment dragger
15f397b
to
c126ed4
Compare
Closes #476