Skip to content
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

Closed
JaynouOliver opened this issue Oct 15, 2023 · 13 comments
Closed

Enable Dark Mode in docs page #3179

JaynouOliver opened this issue Oct 15, 2023 · 13 comments
Labels
Community Issue/PR opened by the open-source community Issue: Feature Request New feature or improvement to existing feature

Comments

@JaynouOliver
Copy link

https://docs.kedro.org/en/stable/#
image

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

@JaynouOliver JaynouOliver added the Issue: Feature Request New feature or improvement to existing feature label Oct 15, 2023
@astrojuanlu astrojuanlu added the Community Issue/PR opened by the open-source community label Oct 15, 2023
@PranavJayachandran
Copy link

I am interested. Can you assign this issue to me?

@astrojuanlu
Copy link
Member

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.

@astrojuanlu
Copy link
Member

astrojuanlu commented Oct 15, 2023

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!

@JaynouOliver
Copy link
Author

Sure, I will a look upon it and start working on it. Thank you for providing me the details.

@JaynouOliver
Copy link
Author

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 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

@stichbury
Copy link
Contributor

stichbury commented Oct 16, 2023

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 kedro-org/kedro-sphinx-theme#5 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.

@JaynouOliver
Copy link
Author

Ok I didn't knew about that. How much time may that take to get deployed to production?

@stichbury
Copy link
Contributor

stichbury commented Oct 16, 2023

@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.

@JaynouOliver
Copy link
Author

So what shall I do? shall I proceeded with the code?

@astrojuanlu
Copy link
Member

Go ahead @JaynouOliver ! As long as we don't change the theme, we'll be happy to have a look at the contribution.

@stichbury
Copy link
Contributor

@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.

@JaynouOliver
Copy link
Author

ok

@stichbury
Copy link
Contributor

Closing this ticket because it will form part of a broader update to the documentation design covered by kedro-org/kedro-sphinx-theme#5 (and if completed standalone would cause more problems than it solves because of #3416).

I will note that dark mode is something to add as a requirement when we cover the redesign.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Issue/PR opened by the open-source community Issue: Feature Request New feature or improvement to existing feature
Projects
Archived in project
Development

No branches or pull requests

4 participants