Skip to content

Conversation

@markuslindmark
Copy link
Contributor

In light theme:

Improve contrast on brand foreground:
colorBrandForeground1
colorBrandForeground2
colorBrandForeground2Hover
colorBrandForeground2Pressed

Improve background contrast on
colorNeutralBackground2

Make colorSubtleBackgroundHover visible on top of all background

Correction of illustration "location" that got themes switched

Describe your changes

Please include a summary of the change, a relevant motivation and context.

Issue ticket number and link

  • Fixes #(issue)

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have verified that the code builds perfectly fine on my local system
  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have verified that my code follows the style already available in the repository
  • I have made corresponding changes to the documentation

In light theme:

Improve contrast on brand foreground:
colorBrandForeground1
colorBrandForeground2
colorBrandForeground2Hover
colorBrandForeground2Pressed

Improve background contrast on
colorNeutralBackground2

Make colorSubtleBackgroundHover visible on top of all background

Correction of illustration "location" that got themes switched
@markuslindmark markuslindmark requested a review from a team as a code owner December 18, 2025 15:37
@markuslindmark markuslindmark enabled auto-merge (squash) December 19, 2025 10:12
Copy link
Contributor

@axis-d0op axis-d0op left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@markuslindmark markuslindmark merged commit fa6e999 into main Dec 19, 2025
10 checks passed
@markuslindmark markuslindmark deleted the improvement-of-color-tokens-contrast branch December 19, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants