-
Notifications
You must be signed in to change notification settings - Fork 369
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 Mode #525
Dark Mode #525
Conversation
I've implemented this functionality into the dark theme, and it does reduce most of the FOIT (though there is still a little bit of FOIT in the background of where the panels are. Additionally, I'd like to implement this into the Random Battles calculator (and perhaps the 1vAll and Allv1 calculators, but that would take a good amount of time to add the class names to the HTML elements. |
Thank you for this, this looks really good, but I don't want to merge this without having support for all modes. Switching between randoms and the normal calc etc and going back and forth between light and dark will not go well. |
Absolutely fine, I'll update when I've finished adding dark mode support across all the modes. |
Has small styling issues that I couldn't figure out how to fix
Ok, I've added dark mode for the other three calculators. I couldn't figure out how to style some of the stuff in the Honkalculator because it was all inserted using jQuery (namely, there's this little piece of text with a search box on the left that I don't know how to define styles for (it has neither a class nor an id): Otherwise, I've realized what the "cascading" in CSS means (aka I've removed the |
perfect, thank you! |
* Finished dark theme * Fixing code for tests; I miss es6 * Reduced FOIT, but is still there * Added dark theme to Dynamax buttons * Removed duplicate classes * Added dark mode to Randoms calc * Removed unnecessary dark-theme class * Added Dark Mode for Honkalculator Has small styling issues that I couldn't figure out how to fix * Styled secret "Search" label
A dark theme is one of the features that is requested in #348, and since it seemed like basic CSS and JS work, I figured I'd give it a go. Here is the result.
Shortcomings
styles.css
) that would serve as the file that gets used as the style that gets loaded in when the web page loads. Then, the user can select a theme from a dropdown menu, and the page would make the styles corresponding to that theme as the newstyles.css
to pull from, then reload the page to reapply the styles.Screenshots
Open me!