-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Feature: Add YUI-style optimisations for compressed CSS output #847
Comments
...does that really provide an enormous speed boost to the loading time ? |
I'd be interested in this. Every little bit helps, though a linter seems a bit out of scope. https://csslint.net/ |
In hindsight, having a linter does seem a bit out of scope, although things like removing units from values can reduce file size by quite a bit if you have a lot of definitions. Stuff like condensing border definitions into one rule is kind of out of the scope of SASS. |
Don't get me wrong, I still very much want that kind of compression and minification, along with linting. In fact, I have been trying to handle this with Grunt or Rake tasks lately, so I'm glad this discussion is happening now (yes, I use and am aware of the compressed style of output). What we're talking about is pushing Sass beyond compression into linting. Maybe it could fit into the goals of Sass, I dunno. I think the discussion would have to be with a wider audience. Maybe it could be an option in Compass? + @chriseppstein |
Linting could also be something that could be a part of a GUI app like Codekit. Thoughts @bdkjones? |
CodeKit already offers the compressed option for Sass output. Is there a good Sass lint tool available? Sent from my iPhone On Aug 21, 2013, at 7:42, Kevin Suttle [email protected] wrote:
|
Ooh now that's a good question. |
I really thought there'd be more traction and opinions on this. A quick search didn't yield much other than the following: |
+1 on remove leading zero. The other optimizations look good as well. :) |
@Undeterminant It looks like the font list compression is already present in master. If I've misunderstood, please add another example and I'll take a look. |
This bug was reported 2 months ago, so, it may have silently been fixed with a code update. Either way, if it's in master, you can just accept it as implemented. |
Any updates re: Sass/SCSS linting? |
Resolves one part of sass#847
Resolves one part of sass#847
Duplicate of #687. |
Just to name a few optimisations that the YUI compressor handles but SASS doesn't:
0.5em
->.5em
)0px
->0
)"Font 1", "Font 2"
->"Font 1","Font 2"
)The text was updated successfully, but these errors were encountered: