-
Notifications
You must be signed in to change notification settings - Fork 104
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
Fix Carbonization #697
Fix Carbonization #697
Conversation
@@ -75,7 +75,7 @@ | |||
--cds-border-subtle, | |||
var(--cds-border-subtle-01, var(--color-grey-225-10-85)) | |||
); | |||
--color-borders-readonly: var(--cds-border-disabled, var(--color-grey-225-10-75)); | |||
--color-borders-readonly: var(--cds-border-subtle, var(--color-grey-225-10-75)); |
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.
@@ -1084,7 +1085,7 @@ const DROPDOWN_STYLES = css` | |||
|
|||
& .fjs-dropdownlist-item:hover, | |||
& .fjs-dropdownlist-item.focused { | |||
background-color: var(--cds-layer-hover, var(--cds-layer-hover-01)); | |||
background-color: var(--cds-layer-hover); |
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.
I also took the opportunity to remove the unnecessary fallbacks
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.
I think while experimenting with the styles, I was able to create setups where the cds-layer-hover
without a specific number wasn't specified, so that's why these fallbacks landed in initially. However, I think I pretty much misconfigured something, so it's very unlikely that this will happen.
0e0a47d
to
a78109c
Compare
a78109c
to
14e5b3d
Compare
I will have a look today 👍 |
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.
LGTM 👍
I added one commit to use the layer-one in our test setup.
Closes camunda/tasklist#2966
form-js
element or visually changes an existing component.