-
Notifications
You must be signed in to change notification settings - Fork 911
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
Support dark mode #331
Comments
Any update on this? Would love that feature to be available 😍 |
Someone could add that feature to Docsy, It seems not to hard to do it: GoogleContainerTools/skaffold@ccdffd1 If I have time, I'll try to code it ⚡ |
Help wanted, that would be pretty epic to see! 🤗 |
I have tried to implement the above mentioned - I would call it - "darkmode hack" ;-) and made a PR with the WIP flag #563 In my opinion, the result generated by the snippets from GoogleContainerTools/skaffold@ccdffd1 is ... - yes it's dark and it's a mode and it's really quick to implement ;-) I think, if we really want a darkmode, it would be better to write an separate _variables.scss to define the colors by hand. Any other comments? |
After reading a little more into the dark mode topic, I'm afraid that a "useful implementation" will require a little more work (and discussion). A few thoughts from me:
That's why I would wait to develop a dark mode with my own stylesheet until Docsy switches to Bottstrap 5 and then use the approach from https://github.com/vinorodrigues/bootstrap-dark-5. If you can live with the above dark mode hack (GoogleContainerTools/skaffold@ccdffd1), you can use my PR #563 as a template. Any thoughts / suggestions / ideas? |
Is there any more need or interest for the PR #563? |
As nobody showed further interest, I'll close this PR #563. |
Thanks a lot for your attempt @narrenfrei - I would definitely love dark mode, but it sounds like separate scss might be a better way 🤔 |
sounds like for properly implementing the dark mode, Docsy would first need to be upgraded to Bootstrap 5? I'm interested in the dark mode and before I go spinning off my own, I wanted to see if this is already in the works! |
jenkins-x/jx-docs#3053 is how it is done on https://jenkins-x.io/ |
Based on So far it is the most voted issue in this repo. Would be nice if Google folks (@LisaFC, @chalin ?) could escalate this internally to give it a proper frontend engineering. |
The latest docsy has been moved to bootstrap5, so can we use https://github.com/vinorodrigues/bootstrap-dark-5 now? |
👀 |
Direct link to the BS page: https://getbootstrap.com/docs/5.3/customize/color-modes/#dark-mode |
I'm doing this to stay up to date, and because I'm looking forward to dark mode support in Docsy v0.10 (which according to google/docsy#331 seems to be almost finished). There are a few small visual changes, which I think are generally an improvement: - Buttons are more square. - Links now have underscores, for accessibility reasons. - The ellipsis after "Read more..." on the home page is gone. I did a quick browse through the website and couldn't find any breaking changes.
With #1981, I consider this implemented. |
Providing a dark version of https://github.com/google/docsy/blob/master/assets/scss/_variables.scss would be great
Edits (@chalin)
Prep
Tasks
[Dark-mode] Fixes for mobile #1917
Adjustments that will be required to Docsy element styling in support of Bootstrap's dark mode:
[BS5.3] Replace
.navbar-dark
by data attribute and other style adjustments #1896Style adjustments in prep for dark mode support #1908
Style adjustments in prep for dark mode support #1908
.td-box-*
:.td-box--white
should be dark-mode compatible #1967The text was updated successfully, but these errors were encountered: