-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Use SASS for compiling lightbox.css #496
base: master
Are you sure you want to change the base?
Conversation
Looks good, let's get it in. |
Use a var like |
Improved styles, fixed a few wrong moments. |
LGTM |
+1 |
This is an old PR that rewrites the CSS with SASS. I wanted to chime in on why it wasn't merged for anyone who comes across it in the future. For this project I've opted to keep the barrier for usage and outside development very low. So this means no precompilers. At one point I had switched to Coffeescript which caused significant confusion for those unfamiliar with it. Also, at this point, SASS is making way for PostCSS. And with the minimal amount of CSS in the project, it isn't worth it IMHO. For features like this that aren't on the roadmap I keep the PRs open so they are visible for others who might be interested in utilizing the code. |
Added sass preprocessor support (issue #495), added
buildsass
grunt task (sass/autoprefixer).