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

Dark theme for opensourcediversity #118

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

siddhanth339
Copy link

A glimpse of the dark theme:

darkTheme1

darkTheme2

darkTheme3

Also, some links were unnecessary so I have fixed them.

@siddhanth339
Copy link
Author

Please review my PR and let me know if any changes are to be made.
Thanks in advance.

@jancborchardt
Copy link
Member

Hi @siddhanth339, good stuff! :) Here’s some points:

What do you think? :)

@siddhanth339
Copy link
Author

Good idea! I am working on it and will make a commit soon. Thanks for your time!

@siddhanth339
Copy link
Author

siddhanth339 commented May 8, 2021

@jancborchardt please review my latest commit 1f42523

  • CSS variables are used and there is only one file "styles.css" for both light and dark themes
  • Toggle for the theme is removed and prefers-color-scheme is used to detect theme
  • Heading is in branded green colour for the dark mode

@siddhanth339
Copy link
Author

@jancborchardt any updates?

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@siddhanth339 nice! Some feedback:

  • Removing the color completely seems a bit harsh, I would say the top part works nice in the green too? And the dividers would be nice with the dash of color as well. What do you think?
  • Can you check the conflicts with current main branch and resolve them? A lot of this might be because of the tabs to spaces conversion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants