-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Migrate GUI colors test to original CSS color format #114475
Migrate GUI colors test to original CSS color format #114475
Conversation
Some changes occurred in HTML/CSS/JS. cc @GuillaumeGomez, @Folyd, @jsha |
1434250
to
85ed2fe
Compare
@@ -68,7 +68,7 @@ | |||
--test-arrow-color: #f5f5f5; | |||
--test-arrow-background-color: rgba(78, 139, 202, 0.2); | |||
--test-arrow-hover-color: #f5f5f5; | |||
--test-arrow-hover-background-color: #4e8bca; | |||
--test-arrow-hover-background-color: rgb(78, 139, 202); |
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 changed the color scheme in this file and in dark.css
to make it closer to --test-arrow-background-color
so it's more obvious they're linked.
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've confirmed that there are the same color. Just written differently.
@bors r+ |
…mpiler-errors Rollup of 5 pull requests Successful merges: - rust-lang#114287 (update overflow handling in the new trait solver) - rust-lang#114475 (Migrate GUI colors test to original CSS color format) - rust-lang#114482 (Fix ui-fulldeps missing the `internal_features` lint on stage 0) - rust-lang#114490 (Fix a typo in the error reporting for sealed traits.) - rust-lang#114491 (Rename issue rust-lang#114423 test files to include context) r? `@ghost` `@rustbot` modify labels: rollup
Follow-up of #111459.
r? @notriddle