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

feat: CSS3 variables & improved CSS dev experience #1748

Draft
wants to merge 99 commits into
base: master
Choose a base branch
from

Conversation

zackspear
Copy link
Contributor

This PR is based off the original work done in #1425. Huge shout out to @BaileyJM02 for the hard work with the initial setup for this.

The purpose of this PR is to lay the ground work for future CSS enhancements and improving the developer experience for styles.

Removing the idea of have multiple CSS files for theming differences. Ideally there's a singular base CSS file (should work for both layout types) with all colors using CSS variables. And the theme files are used to solely set CSS variables for color differences, instead of copied and pasted styles with different colors. Less duplicated code should make maintaining and adding new styles easier while also being less prone to errors.

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.

3 participants