-
Notifications
You must be signed in to change notification settings - Fork 11.2k
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
Issue 105 : Dark Mode + Theming #118 #119
Conversation
* Themes scss file created with default color scheme and dark color scheme. * Logic for managing dark mode toggle added. * Modified existing scss/html to work with theme toggling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Looks great, especially the theming feature.
One request before I merge: could you make the dark mode an optional feature (off by default)?
To do that, simply add a variable to _config.yml
called enable_dark_mode
(similar to other optional features) and make sure that dark-mode specific code is loaded only when enable_dark_mode=true
.
Sounds good I'll make the change by Wednesday latest. |
* Added the requested option of disabling/enabling dark mode via config files.
Made the requested change. If you require a rebased version I can make another pull request using the master branch : https://github.com/Jongwoo-Shim/al-folio/tree/master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks for making the changes!
* 'master' of https://github.com/alshedivat/al-folio: Minor fixes of the dark mode + automatic equation numbering (alshedivat#160) Update text color in navbar in dark mode (alshedivat#154) add wide screen configuration (alshedivat#157) Support more accurate author annotation (alshedivat#156) Update README, change theme-color in _themes.scss (alshedivat#153) Move GA script to head.html (alshedivat#140) Update _base.scss (alshedivat#137) Adding back the description on the About page (alshedivat#136) Add keybase badge (alshedivat#132) add website (alshedivat#134) Issue 105 : Dark Mode + Theming alshedivat#118 (alshedivat#119) Updated Google Analytics code (alshedivat#123) Github MetaData (alshedivat#121) Added Twitter Card Support (alshedivat#120) Fix issues with loading scripts (alshedivat#117) Switch to MathJax 3.x. Fixes alshedivat#115. Update README.md Update README.md # Conflicts: # _config.yml # _includes/head.html # _includes/header.html # _sass/_base.scss # _sass/_variables.scss
* Dark Mode + Theming * Themes scss file created with default color scheme and dark color scheme. * Logic for managing dark mode toggle added. * Modified existing scss/html to work with theme toggling. * Fixed hover issues * Added enable_darkmode to configs. * Added the requested option of disabling/enabling dark mode via config files.
* Dark Mode + Theming * Themes scss file created with default color scheme and dark color scheme. * Logic for managing dark mode toggle added. * Modified existing scss/html to work with theme toggling. * Fixed hover issues * Added enable_darkmode to configs. * Added the requested option of disabling/enabling dark mode via config files.
* Dark Mode + Theming * Themes scss file created with default color scheme and dark color scheme. * Logic for managing dark mode toggle added. * Modified existing scss/html to work with theme toggling. * Fixed hover issues * Added enable_darkmode to configs. * Added the requested option of disabling/enabling dark mode via config files.
Created separate branch for dark mode, as I maybe submitting another PR.