-
Notifications
You must be signed in to change notification settings - Fork 906
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
Enable Dark Mode in docs page #3179
Comments
I am interested. Can you assign this issue to me? |
Hi @JaynouOliver, @PranavJayachandran, thanks for your interest! Note that Kedro uses sphinx-rtd-theme at the moment, so for us to have a dark mode we would have to either
Feel free to work on any of these, no need to get assigned to it. |
Finally, in our opinion this is a difficult task so in principle we are not labeling it for Hacktoberfest. Unless we see a mostly-ready pull request on time, of course! |
Sure, I will a look upon it and start working on it. Thank you for providing me the details. |
Sure, I am not doing this for the sake of Hacktoberfest, nor did I mention it anywhere to get the issue labelled. I was actually going through the docs when I struck upon this idea, thanks a lot |
I'm really pleased to see this. However, one warning I should give you is that we have some outstanding design changes to come in the docs. The ticket is still awaiting our team to look at it #4257 but there will be a redesign in future, so your work could be short-lived if we move to a different theme or even build tool when it is addressed. |
Ok I didn't knew about that. How much time may that take to get deployed to production? |
@JaynouOliver At present, it's not on our priority list (based on feedback from user research). We have a Kedro 0.19 release to come out first and then we'll circle back to docs redesign. So there will not be any updates to switch away from our current theme in 2023. We could take in an update for dark mode in the short-term. That probably isn't very helpful since it means you risk doing work now that will be brought into Kedro but may drop out again when we do decide how to proceed with a new theme/updates to current them/updates to toolchain. However, I would say that the Sphinx RTD theme maintainers would probably still appreciate a dark mode contribution because regardless of whether Kedro continues to use it, others will. |
So what shall I do? shall I proceeded with the code? |
Go ahead @JaynouOliver ! As long as we don't change the theme, we'll be happy to have a look at the contribution. |
@JaynouOliver I can't really decide for you because I can't commit your time 😄 but if you want to make a contribution for dark mode then it would be welcome. We could potentially use it until such time as we redesign, and maybe after, but cannot guarantee it. |
ok |
https://docs.kedro.org/en/stable/#
It is quite difficult to view the page in light mode in dark. Although we all use an extension but it is better to have a dark mode button that can switch the color
The text was updated successfully, but these errors were encountered: