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

Update GitHub theme, add dark mode #1918

Merged
merged 2 commits into from
Jan 24, 2023

Conversation

dunkmann00
Copy link
Contributor

This updates the GitHub theme and also adds support for a dark mode.

The color values are sourced from the primer/primitives repository. https://github.com/primer/primitives

This resolves #1906

This updates the GitHub theme and also adds support for a dark mode.

The color values are sourced from the primer/primitives repository.
https://github.com/primer/primitives
@tancnle tancnle self-requested a review January 24, 2023 10:27
@tancnle tancnle added the needs-review The PR needs to be reviewed label Jan 24, 2023
Copy link
Collaborator

@tancnle tancnle left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @dunkmann00 ❤️ The PR looks good overall. I have added some suggestions for your consideration. It would be nice to set the background colour correctly when viewing via the visual tool http://localhost:9292/?theme=github.dark

Dark Light
Screenshot 2023-01-24 at 9 43 42 pm Screenshot 2023-01-24 at 9 43 29 pm

Please let me know what you think.

lib/rouge/themes/github.rb Outdated Show resolved Hide resolved
lib/rouge/themes/github.rb Show resolved Hide resolved
lib/rouge/themes/github.rb Show resolved Hide resolved
@dunkmann00
Copy link
Contributor Author

Hi @tancnle! Thanks for taking a look at this and for offering some suggestions about the background. I've gone ahead and added a background color to the text for both modes.

@tancnle
Copy link
Collaborator

tancnle commented Jan 24, 2023

Thanks, @dunkmann00. Look awesome! 🚀

@tancnle tancnle merged commit 6f5aecd into rouge-ruby:master Jan 24, 2023
@dunkmann00 dunkmann00 deleted the update-github-theme branch February 2, 2023 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review The PR needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Github Dark style
2 participants