-
Notifications
You must be signed in to change notification settings - Fork 67
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
fix: change to select colors via token #2247
Conversation
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🔴 | Statements | 2.9% | 102/3512 |
🔴 | Branches | 3.09% | 69/2235 |
🔴 | Functions | 1.44% | 17/1179 |
🔴 | Lines | 2.96% | 102/3448 |
Test suite run success
20 tests passing in 4 suites.
Report generated by 🧪jest coverage report action from 5a35e1c
…ed background color
@yomybaby In a previous commit, I had a misunderstanding about the theme.json file. now it's fixed |
@@ -129,11 +126,14 @@ export const BackendAiStyles = [ | |||
--general-progress-bar-bg: var(--token-colorBorderSecondary, #e8e8e8); | |||
--general-progress-bar-reserved: linear-gradient( | |||
to left, | |||
#722cd7, | |||
#5c7cfa | |||
var(--token-blue-8) 0%, |
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.
@yomybaby
This is the part that controls the gradient color of the progress bar.
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 pushed a commit to remove the gradient color of the progress bar.
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.
LGTM!
This PR Resolves #2246 issue.
Feature
Reflected request changes
Checklist: (if applicable)