Skip to content
This repository has been archived by the owner on Nov 26, 2024. It is now read-only.

Revert "Adding brand colors to CSSinJS exports" #589

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

georgewrmarshall
Copy link
Collaborator

@georgewrmarshall georgewrmarshall commented Jan 19, 2024

Description

This PR reverts the recent updates introducing the brandColors object. The decision to revert these changes comes from the need to release other color and accessibility updates without exposing the brandColors API prematurely. This approach will prevent us from introducing a breaking change in the near future.

While these updates aimed to improve the intuitiveness and ease of use of the brandColors object, we've decided to postpone these changes to ensure a smooth transition and to provide more time for thorough testing and review.

We plan to reintroduce these changes in a future release, taking into account the valuable feedback and discussions that have arisen from this work. We appreciate your understanding and invite further discussion on how we can improve our color system.

@georgewrmarshall georgewrmarshall self-assigned this Jan 19, 2024
@georgewrmarshall georgewrmarshall added code Issue related to code work team-design-system All issues relating to design system labels Jan 19, 2024
@georgewrmarshall georgewrmarshall marked this pull request as ready for review January 19, 2024 18:32
@georgewrmarshall georgewrmarshall requested a review from a team as a code owner January 19, 2024 18:32
@georgewrmarshall georgewrmarshall removed the code Issue related to code work label Jan 19, 2024
@georgewrmarshall georgewrmarshall merged commit 0910f70 into main Jan 19, 2024
7 checks passed
@georgewrmarshall georgewrmarshall deleted the revert-582-fix/581/brand-colors-css-in-js branch January 19, 2024 20:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-design-system All issues relating to design system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants