-
Notifications
You must be signed in to change notification settings - Fork 110
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
Minor color improvements #522
Conversation
shanecmiller23
commented
Aug 8, 2023
•
edited
Loading
edited
- Improve contrast for accordions/expandable blocks
- Improve coloring for "Join the Crossplane Slack" home page tooltip
- Replaced all old color variables with the new palette variables
- Fix dark mode coloring for vertical striped tables
- Remove table aqua background hover states
Signed-off-by: Shane Miller <shanecm.ux@gmail.com>
✅ Deploy Preview for crossplane ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@jbw976 per some of your recent feedback, darkened the borders/icons for these components |
Signed-off-by: Shane Miller <shanecm.ux@gmail.com>
Signed-off-by: Shane Miller <shanecm.ux@gmail.com>
@@ -51,7 +51,7 @@ | |||
|
|||
// Override Bootstrap accordion colors in the body | |||
.accordion { | |||
--bs-accordion-border-color: #{$fog-800}; | |||
--bs-accordion-border-color: #{$fog-500}; |
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.
There's something fishy going on with the bottom rounded corners of the accordions when they are in their expanded state - looks like it's still a dark color that doesn't match the straight line edge borders, or they are clipped or something? they look like they are missing essentially - do you see this too?
e.g. on https://deploy-preview-522--crossplane.netlify.app/v1.13/concepts/compositions/
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 see the same thing in dark mode.
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.
Signed-off-by: Shane Miller <shanecm.ux@gmail.com>
Signed-off-by: Shane Miller <shanecm.ux@gmail.com>