Skip to content

feat: update internal brand colours and use cases#2363

Open
frshwtr wants to merge 7 commits intomainfrom
or-1805/update-internal-brand-colours
Open

feat: update internal brand colours and use cases#2363
frshwtr wants to merge 7 commits intomainfrom
or-1805/update-internal-brand-colours

Conversation

@frshwtr
Copy link
Contributor

@frshwtr frshwtr commented Feb 13, 2026

No description provided.

@frshwtr frshwtr marked this pull request as ready for review February 26, 2026 15:10
@frshwtr frshwtr requested a review from a team as a code owner February 26, 2026 15:10
'o3-type-label-text-case': uppercase,
'_o3-form-input-border': 1.5px solid #b3b3b3,
'_o3-form-input-border-error': 1.5px solid #cc0000,
'_o3-form-input-border-error': 1.5px solid [object Object],
Copy link
Member

Choose a reason for hiding this comment

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

issue: [object Object]

--_o3-form-color-use-case-background-error: rgba(204, 0, 0, 0.06);
--_o3-form-color-use-case-border-disabled: #9ec0bd;
--_o3-form-input-border: 1.5px solid #b3b3b3;
--_o3-form-input-border-error: 1.5px solid [object Object];
Copy link
Member

Choose a reason for hiding this comment

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

issue: [object Object]

"error": {
"value": {
"color": "{o3.color.use-case.error}",
"color": "{o3.color.use-case.error.foreground}",
Copy link
Member

Choose a reason for hiding this comment

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

question: Was this change done in token studio?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@joelcarr In this case I manually edited the value as it was known to me. Here's a screenshot showing the change pulls the correct value in tokens studio
image

@joelcarr
Copy link
Member

As discussed in slack, this will be a breaking change. Please can you update the MIGRATION.md doc to highlight what the breaking change is?

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.

3 participants