Skip to content

Conversation

adids1221
Copy link
Contributor

@adids1221 adids1221 commented Aug 20, 2025

Description

ℹ️ Color accessibility fixes
Update tokens system colors as part of A11Y effort.

  • blue30 hex color
  • text/icon Success update from green10 -> green5
  • text/icon Major update from orange10 -> orange5
  • text/icon Danger update from red10 -> red5

Test and snapshot updates.

Changelog

Color accessibility fixes, update primary, success, danger, major tokens colors.

Additional info

Ticket 4802

@adids1221 adids1221 requested a review from nitzanyiz August 21, 2025 07:57
@adids1221 adids1221 marked this pull request as draft August 25, 2025 13:39
@adids1221 adids1221 marked this pull request as ready for review August 27, 2025 11:04
@adids1221
Copy link
Contributor Author

@nitzanyiz I spoke with Ethan about that, since this is part of a11y effort we will keep this fix in public.

@adids1221 adids1221 added the Important for Next Release PR that must be included in the release version label Aug 27, 2025
@nitzanyiz nitzanyiz assigned adids1221 and unassigned nitzanyiz Aug 28, 2025
@adids1221 adids1221 requested a review from nitzanyiz August 28, 2025 08:20
@adids1221 adids1221 assigned nitzanyiz and unassigned adids1221 Aug 28, 2025
@adids1221 adids1221 added hotfix Requires a hotfix release and removed Important for Next Release PR that must be included in the release version labels Aug 28, 2025
@adids1221 adids1221 merged commit 2cb597d into master Aug 28, 2025
1 check passed
@adids1221 adids1221 deleted the fix/update_blue30_system_color_a11y branch August 28, 2025 08:25
adids1221 added a commit that referenced this pull request Aug 28, 2025
* update: blue 30 system color hex, test fix

* update: textSuccess, iconSuccess set to green5

* update: textDanger, iconDanger set to red5

* update: change textMajor and iconMajor colors to orange5

* update: change blue30 color hex to #0F62E6 and update related tests

* Update src/style/__tests__/colors.spec.js

Co-authored-by: Nitzan Yizhar <[email protected]>

---------

Co-authored-by: Nitzan Yizhar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix Requires a hotfix release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants