-
Notifications
You must be signed in to change notification settings - Fork 22
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
docs: Change colors of links #1295
Conversation
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.
Thinking a bit more about the problem, I think that we should instead overload the color
define in the theme: https://github.com/pydata/pydata-sphinx-theme/blob/3d03aee9088e03ccacff320a836889cb8396d801/src/pydata_sphinx_theme/assets/styles/variables/_color.scss#L285-L286
As far I see, we should have to redefine:
--pst-color-link
--pst-color-link-hover
--pst-color-inline-code-links
So it means that any CSS rule using those colors will then be consistent.
That's why you better redefined the color rather than modify the HTML objects. |
Yes, it's a draft, it's not working the way I want for now. I think I'll spend another hour trying, and then stop, because I don't think it's worth it. @glemaitre, when you say "That's why you better redefined the color rather than modify the HTML objects.", do you mean that these colors are instantiated somewhere and I can modify them? That would make things much easier! Edit: ok, I found it. And after I read that you had the solution Guillaume 😅 |
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.
Thanks! Indeed CSS is the right way to do this IMO.
Indeed, it is what I have in mind. You need to check the rendering but it was the idea. |
yes, the colors for light mode are not exactly those of the logo because the contrast was not nearly high enough, but it's the same idea, orange and blue, only darker. |
Closes #1008