-
Notifications
You must be signed in to change notification settings - Fork 873
[Colors] Create text-ink and text-ghost tokens #9379
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
Changes from all commits
5daa3d5
9bef391
26254b6
7955fb0
e058a61
0509b65
1413552
670265d
2f9edd1
1c00d97
9a30969
c5ca2bc
86f5e1d
61074c5
4a9d892
0522786
a0ca2b5
3171c50
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| - Added `textInk` and `textGhost` color tokens for text and icon colors that should always remain dark or light regardless of color mode. | ||
|
|
||
| **Breaking changes** | ||
|
|
||
| - Removed `ink` and `ghost` theme tokens. Use `textInk` / `textGhost` for text and icon colors or `plainDark` /`plainLight` for non-text use cases. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -354,8 +354,6 @@ | |
| "euiColorSeverityDanger": "#EE4C48", | ||
| "euiColorChartLines": "#7186A8", | ||
| "euiColorChartBand": "#243147", | ||
| "euiColorGhost": "#FFFFFF", | ||
| "euiColorInk": "#000000", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Blocking: We'll need to add the new tokens If I see it correctly, then these should use the text JSON tokens?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I treated the JSON in a similar way like Sass but you are absolutely right we should add them here. I will go through the Kibana usage again and correct as needed.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just as additional context: JSON tokens used to be mapped from Sass and the original intention was to remove them as well but while rolling out Borealis we noticed that we need to support them for static use cases where the react theme is not available 🫠
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| "euiColorPrimary": "#61A2FF", | ||
| "euiColorAccent": "#EE72A6", | ||
| "euiColorAccentSecondary": "#16C5C0", | ||
|
|
@@ -397,6 +395,8 @@ | |
| "euiColorTextSubdued": "#98A8C3", | ||
| "euiColorTextDisabled": "#6A7FA0", | ||
| "euiColorTextInverse": "#07101F", | ||
| "euiColorTextGhost": "#FFFFFF", | ||
|
mgadewoll marked this conversation as resolved.
|
||
| "euiColorTextInk": "#07101F", | ||
| "euiColorBackgroundBasePrimary": "#0A2342", | ||
| "euiColorBackgroundBaseAccent": "#351725", | ||
| "euiColorBackgroundBaseAccentSecondary": "#03282B", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| - Added `textInk` and `textGhost` color tokens for text and icon colors that should always remain dark or light regardless of color mode. | ||
|
|
||
| **Breaking changes** | ||
|
|
||
| - Removed `ink` and `ghost` theme tokens. Use `textInk` / `textGhost` for text and icon colors or `plainDark` /`plainLight` for non-text use cases. |
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 just realized that this PR had the
skip-changelog-euilabel. I missed that in the review 🫠We should add a changelog for this on
euias well as the tokens are consumed through the theme fromeui.I added a PR for it here.