Merge branch 'master' into text-indent

Author: stubbornella

.gitignore

@@ -1,2 +1 @@
 csslint.pnproj
-build/

CHANGELOG

@@ -0,0 +1,22 @@
+Next (not yet released)
+
+* Rhino and Node CLIs both exit with code 1 when there are errors (pull #72)
+* Changed description of adjoining classes to be unsupported in IE6 (fixes #11)
+* Made license comment important in build files (fixes #46)
+* Code cleanup (pull #70)
+* Cleanup of whitespace (pull #74)
+
+June 18, 2011 - v0.2.0
+
+* Make sure headings are counted correctly (fixes #25)
+* Make sure float: none doesn't count as a float for rules that care (fixes #10)
+* Fix erroneous missing standard border warning (fixes #7)
+* Ensure width: 100% is okay when box-sizing is specified (fixes #5)
+* Fixed up width/height properties for box model rule (fixes #8)
+* Incorporated Rhino CLI and updated contributors list
+* Include build file in Git repo so people can grab directly (fixes #59)
+* Added rule for tracking using of !important from shinuza (fixes #50)
+
+June 15, 2011 - v0.1.0
+
+* Initial release

LICENSE

@@ -1,20 +1,20 @@
-CSSLint
-Copyright (c) 2011 Nicole Sullivan and Nicholas C. Zakas. All rights reserved.
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
+CSSLint
+Copyright (c) 2011 Nicole Sullivan and Nicholas C. Zakas. All rights reserved.
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

README.md

@@ -12,7 +12,7 @@ By default, CSSLint shows any parsing errors. Parsing errors usually mean you mi
 
 ### Don't use adjoining classes
 
-Adjoining classes look like `.foo.bar`. While technically allowed in CSS, these aren't handled properly by Internet Explorer 7 and earlier.
+Adjoining classes look like `.foo.bar`. While technically allowed in CSS, these aren't handled properly by Internet Explorer 6 and earlier.
 
 ### Remove empty rules
 
@@ -36,6 +36,10 @@ Even though you can define any group of properties together in a CSS rule, some
 
 Removed the ignored or problematic properties decreases file size and improves performance.
 
+### Avoid using to many !important declarations
+
+Using `!important` overides any cascaded rule and may lead to specificity war. CSSLint checks if you've used `!important`, and if so, displays a warning. If there's at least 10 `!important` declaration in your code CSSLint displays an error.
+
 ### Don't use too many floats
 
 Using `float` for layout isn't a great idea, but sometimes you have to. CSSLint simply checks to see if you've used `float` more than 10 times, and if so, displays a warning. Using this many floats usually means you need some sort of abstraction to achieve the layout.
@@ -70,4 +74,43 @@ Heading elements (`h1`-`h6`) should have exactly one rule on a site. CSSLint war
 
 ### Be careful using width: 100%
 
-Using `width: 100%` on an element whose parent element has padding will result in the child stretching outside of the parent's bounding box. It's generally not a good idea to use `width: 100%`. Instead, use `width: auto` or `display: block`.
+Using `width: 100%` on an element whose parent element has padding will result in the child stretching outside of the parent's bounding box. It's generally not a good idea to use `width: 100%`. Instead, use `width: auto` or `display: block`.
+
+### Zero values don't need units
+
+An easy way to save bytes in CSS is not include units when a value is 0. For instance, `0px` and `0` are the exact same measurement, so leave off the units and save!
+
+### Vendor prefixed properties should also have the standard
+
+When using vendor-prefixed properties such as `-moz-border-radius`, make sure to also include the standard property. The standard property should preferably come after the vendor-prefixed one, such as:
+
+```css
+.foo {
+    -moz-border-radius: 5px;
+    border-radius: 5px;
+}
+```
+
+### CSS gradients require all browser prefixes
+
+Right now, there is no standard CSS gradient implementation, which means using CSS gradients in a cross-browser way requires using many different vendor-prefixed versions. CSSLint warns when a rule with a CSS gradient doesn't have gradients for all supporting browsers. 
+
+### Avoid selectors that look like regular expressions
+
+CSS3 adds complex attribute selectors such as `~=` that are slow. When using attribute selectors, don't use the complex equality operators to avoid performance penalties.
+
+### Beware of broken box models
+
+Borders and padding add space outside of an element's content. Setting `width` or `height` along with borders and padding is usually a mistake because you won't get the visual result you're looking for. CSSLint warns when a rule uses `width` or `height` in addition to padding and/or border.
+
+## Contributors
+
+### Creators
+
+1. Nicole Sullivan, http://www.stubbornella.org
+1. Nicholas C. Zakas, http://www.nczonline.net
+
+### Contributors
+
+1. Samori Gorse, https://twitter.com/shinuza (Rules, Non-zero Exit Code for CLI)
+1. Eitan Konigsburg, https://twitter.com/eitanmk (Rhino CLI)

build.xml

@@ -3,6 +3,7 @@
     <!-- the directories containing the source files -->
     <property name="src.dir" value="./src" />
     <property name="npm.dir" value="./npm" />
+    <property name="tests.dir" value="./tests" />
 
     <!-- the directories and files to output to -->
     <property name="build.dir" value="./build" />
@@ -15,6 +16,7 @@
     <property name="core.build.file" value="csslint.js"/>
     <property name="node.build.file" value="csslint-node.js"/>
     <property name="worker.build.file" value="csslint-worker.js"/>
+    <property name="tests.build.file" value="csslint-tests.js"/>
     <property name="rhino.build.file" value="csslint-rhino.js"/>
 
     <loadfile property="license.text" srcfile="LICENSE" />   
@@ -27,7 +29,7 @@
     <!-- build the core library -->
     <target name="build.core">    
         <concat destfile="${build.dir}/${core.build.file}"  fixlastline="true">
-            <header trimleading="yes">/* 
+            <header trimleading="yes">/*! 
             ${license.text}
             */
             var CSSLint = (function(){
@@ -47,7 +49,7 @@
     <!-- build the web worker library -->
     <target name="build.worker">    
         <concat destfile="${build.dir}/${worker.build.file}"  fixlastline="true">
-            <header trimleading="yes">/* 
+            <header trimleading="yes">/*!
             ${license.text}
             */
             </header>
@@ -64,7 +66,7 @@
     <target name="build.node">
 
         <concat destfile="${build.dir}/${node.build.file}"  fixlastline="true">
-            <header trimleading="yes">/* 
+            <header trimleading="yes">/*!
             ${license.text}
             */
             </header>
@@ -85,7 +87,25 @@
 
     </target>
 
+    <!-- build the tests into a single file -->
+    <target name="build.tests">    
+        <concat destfile="${build.dir}/${tests.build.file}"  fixlastline="true">
+            <fileset dir="${tests.dir}/rules" includes="*.js" />           
+        </concat>        
+    </target>   
+
+
+    <!-- build for rhino CLI integration -->
+    <target name="build.rhino" depends="build.core">
+        <concat destfile="${build.dir}/${rhino.build.file}" fixlastline="true">
+            <filelist dir="${build.dir}" files="${core.build.file}" />
+            <filelist dir="${src.dir}/rhino" files="cli.js" />
+        </concat>
+    </target>
+    
+    
+    
     <!-- Build all files -->
-    <target name="all" depends="clean,build.core,build.worker,build.node"/>
+    <target name="all" depends="clean,build.core,build.worker,build.node,build.tests,build.rhino"/>
 
 </project>