Add simple lesshint support

emmanuj · Nov 26, 2015 · 8d36dc6 · 8d36dc6
1 parent 8a1736c
commit 8d36dc6
Show file tree

Hide file tree

Showing 47 changed files with 290 additions and 194 deletions.
diff --git a/.lesshintrc b/.lesshintrc
@@ -0,0 +1,12 @@
+{
+  "propertyOrdering": false,
+  "hexLength": "short",
+  "stringQuotes": false,
+  "decimalZero": false,
+  "importantRule": false,
+  "zeroUnit": "no_unit",
+  "qualifyingElement": false,
+  "duplicateProperty": false,
+  "importPath": false,
+  "finalNewline": false
+}
diff --git a/package.json b/package.json
@@ -84,6 +84,7 @@
     "eslint": "^1.1.0",
     "eslint-config-airbnb": "^1.0.0",
     "eslint-plugin-babel": "^2.1.1",
+    "eslint-plugin-markdown": "git+https://github.com/ant-design/eslint-plugin-markdown.git",
     "eslint-plugin-react": "^3.3.1",
     "expect.js": "~0.3.1",
     "extract-text-webpack-plugin": "^0.9.1",
@@ -93,14 +94,14 @@
     "json-loader": "^0.5.1",
     "less": "~2.5.3",
     "less-loader": "^2.2.0",
+    "lesshint": "git+https://github.com/ant-design/lesshint.git",
     "lodash": "^3.10.0",
     "nico-jsx": "~0.6.0",
     "pre-commit": "1.x",
     "react": "~0.14.2",
     "react-addons-test-utils": "~0.14.2",
     "react-dom": "~0.14.2",
     "react-router": "~1.0.0",
-    "eslint-plugin-markdown": "git+https://github.com/ant-design/eslint-plugin-markdown.git",
     "webpack": "^1.10.1",
     "webpack-babel-jest": "^1.0.0",
     "webpack-dev-middleware": "^1.2.0"
@@ -111,8 +112,9 @@
     "clean": "rm -rf _site dist",
     "deploy": "rm -rf node_modules && node scripts/install.js && npm run just-deploy",
     "just-deploy": "npm run clean && webpack --config webpack.deploy.config.js && webpack --config webpack.antd.config.js && NODE_ENV=PRODUCTION nico build && node scripts/deploy.js",
-    "lint": "eslint components test index.js --ext '.js,.jsx' && npm run mdlint",
+    "lint": "eslint components test index.js --ext '.js,.jsx' && npm run mdlint && npm run lesshint",
     "mdlint": "eslint components/*/demo/*.md --ext '.md' --global 'React,ReactDOM' --rule 'no-console: 0'",
+    "lesshint": "lesshint style/ -e 'style/+(core|mixins)/+(base|iconfont|normalize|layouts|compatibility|grid).less'",
     "test": "npm run lint && webpack && npm run jest",
     "jest": "jest",
     "pub": "sh ./scripts/publish.sh",

diff --git a/style/components/alert.less b/style/components/alert.less
@@ -55,7 +55,8 @@
     }
   }
 
-  &-close-icon, &-with-description-close-icon {
+  &-close-icon,
+  &-with-description-close-icon {
     font-size: 12px;
     position: absolute;
     right: 8px;

diff --git a/style/components/badge.less b/style/components/badge.less
@@ -24,7 +24,8 @@
     z-index: 10;
     font-family: tahoma;
     box-shadow: 0 0 0 1px #fff;
-    a, a:hover {
+    a,
+    a:hover {
       color: #fff;
     }
   }
@@ -63,9 +64,9 @@ a .@{badge-prefix-cls} {
   }
 }
 
-.@{number-prefix-cls}{
+.@{number-prefix-cls} {
   overflow: hidden;
-  &-only{
+  &-only {
     display: inline-block;
     transition: transform .3s @ease-in-out;
   }

diff --git a/style/components/button.less b/style/components/button.less
@@ -34,7 +34,8 @@
     .btn-ghost;
   }
 
-  &-circle, &-circle-outline {
+  &-circle,
+  &-circle-outline {
     .btn-circle(@btn-prefix-cls);
   }
 
@@ -79,7 +80,8 @@
   }
 
   // To ensure that a space will be placed between character and `Icon`.
-  > .@{iconfont-css-prefix} + span, > span + .@{iconfont-css-prefix} {
+  > .@{iconfont-css-prefix} + span,
+  > span + .@{iconfont-css-prefix} {
     margin-left: 0.5em;
   }
 }
diff --git a/style/components/calendar.less b/style/components/calendar.less
@@ -47,8 +47,10 @@
     height: 246px;
   }
 
-  table, td, th, td {
-    border: none;
+  table,
+  th,
+  td {
+    border: 0;
   }
 
   td {
@@ -150,7 +152,7 @@
   }
 
   &-fullscreen {
-    border-top: none;
+    border-top: 0;
   }
 
   &-fullscreen &-table {

diff --git a/style/components/carousel/slick-theme.less b/style/components/carousel/slick-theme.less
@@ -16,9 +16,10 @@
     top: 50%;
     margin-top: -10px;
     padding: 0;
-    border: none;
+    border: 0;
     outline: none;
-    &:hover, &:focus {
+    &:hover,
+    &:focus {
       outline: none;
       background: transparent;
       color: transparent;
@@ -79,7 +80,8 @@
         color: transparent;
         padding: 5px;
         cursor: pointer;
-        &:hover, &:focus {
+        &:hover,
+        &:focus {
           outline: none;
           &:before {
             opacity: 1;

diff --git a/style/components/collapse.less b/style/components/collapse.less
@@ -1,12 +1,10 @@
 @collapse-prefix-cls: ant-collapse;
 
-#arrow {
-  .close() {
-    .iconfont-size-under-12px(7px, 270deg);
-  }
-  .open() {
-    .iconfont-size-under-12px(7px, 360deg);
-  }
+.collapse-close() {
+  .iconfont-size-under-12px(7px, 270deg);
+}
+.collapse-open() {
+  .iconfont-size-under-12px(7px, 360deg);
 }
 
 .@{collapse-prefix-cls} {
@@ -17,7 +15,7 @@
   & > &-item {
     border-top: 1px solid #d9d9d9;
     &:first-child {
-      border-top: none;
+      border-top: 0;
     }
 
     > .@{collapse-prefix-cls}-header {
@@ -29,7 +27,7 @@
       position: relative;
 
       .arrow {
-        #arrow > .close();
+        .collapse-close();
         .iconfont-mixin();
         position: absolute;
         color: #666;
@@ -44,7 +42,7 @@
         top: ~"16px \9";
         left: ~"0 \9";
         &:before {
-          content:"\e611";
+          content: "\e611";
         }
       }
     }
@@ -80,7 +78,7 @@
 
   & > &-item > &-header[aria-expanded="true"] {
     .arrow {
-      #arrow > .open();
+      .collapse-open();
     }
   }
 }
diff --git a/style/components/datepicker/Calendar.less b/style/components/datepicker/Calendar.less
@@ -115,8 +115,10 @@
     width: 100%;
   }
 
-  table, td, th, td {
-    border: none;
+  table,
+  th,
+  td {
+    border: 0;
   }
 
   &-calendar-table {
@@ -206,7 +208,8 @@
     border-bottom-right-radius: 4px;
   }
 
-  &-last-month-cell &-date, &-next-month-btn-day &-date {
+  &-last-month-cell &-date,
+  &-next-month-btn-day &-date {
     color: #bbb;
   }
 

diff --git a/style/components/datepicker/DecadePanel.less b/style/components/datepicker/DecadePanel.less
@@ -57,8 +57,9 @@
   }
 }
 
-.@{calendar-prefix-cls}-decade-panel-last-century-cell, .@{calendar-prefix-cls}-decade-panel-next-century-cell {
-  .@{calendar-prefix-cls}-decade-panel-decade{
+.@{calendar-prefix-cls}-decade-panel-last-century-cell,
+.@{calendar-prefix-cls}-decade-panel-next-century-cell {
+  .@{calendar-prefix-cls}-decade-panel-decade {
     user-select: none;
     -webkit-user-select: none;
   }

diff --git a/style/components/datepicker/YearPanel.less b/style/components/datepicker/YearPanel.less
@@ -57,8 +57,9 @@
   }
 }
 
-.@{calendar-prefix-cls}-year-panel-last-decade-cell, .@{calendar-prefix-cls}-year-panel-next-decade-cell {
-  .@{calendar-prefix-cls}-year-panel-year{
+.@{calendar-prefix-cls}-year-panel-last-decade-cell,
+.@{calendar-prefix-cls}-year-panel-next-decade-cell {
+  .@{calendar-prefix-cls}-year-panel-year {
     user-select: none;
     -webkit-user-select: none;
   }

diff --git a/style/components/dialog.less b/style/components/dialog.less
@@ -1,7 +1,7 @@
 @dialog-prefix-cls: ant-modal;
 
-@import "./dialog/Dialog.less";
-@import "./dialog/Mask.less";
+@import "./dialog/Dialog";
+@import "./dialog/Mask";
 
 .@{dialog-prefix-cls} {
   &-header {

diff --git a/style/components/dialog/Dialog.less b/style/components/dialog/Dialog.less
@@ -25,7 +25,7 @@
   &-content {
     position: relative;
     background-color: #fff;
-    border: none;
+    border: 0;
     border-radius: @border-radius-base;
     background-clip: padding-box;
     outline: 0;

diff --git a/style/components/form.less b/style/components/form.less
@@ -86,7 +86,8 @@ input[type="checkbox"] {
   }
 }
 
-.@{css-prefix}form-text, .@{css-prefix}form-split {
+.@{css-prefix}form-text,
+.@{css-prefix}form-split {
   margin: 0!important;
   padding-top: 7px;
   padding-bottom: 7px;
@@ -243,7 +244,8 @@ form {
   }
 }
 
-.@{css-prefix}form-horizontal, .@{css-prefix}form-inline {
+.@{css-prefix}form-horizontal,
+.@{css-prefix}form-inline {
   label {
     padding-top: 7px;
     padding-bottom: 7px;
@@ -255,7 +257,10 @@ form {
 }
 
 // Validation state
-.has-success, .has-warning, .has-error, .is-validating {
+.has-success,
+.has-warning,
+.has-error,
+.is-validating {
   &.has-feedback:after {
     position: absolute;
     bottom: 0;
@@ -382,36 +387,36 @@ form {
 .is-validating {
   &.has-feedback:after {
     display: inline-block;
-    .animation(loadingCircle 1s infinite linear );
-    content:"\e6a1";
+    .animation(loadingCircle 1s infinite linear);
+    content: "\e6a1";
   }
 }
 
 // need there different zoom animation
 // otherwise won't trigger anim
 @keyframes diffZoomIn1 {
-  0%{
+  0% {
     transform: scale(0);
   }
-  100%{
+  100% {
     transform: scale(1);
   }
 }
 
 @keyframes diffZoomIn2 {
-  0%{
+  0% {
     transform: scale(0);
   }
-  100%{
+  100% {
     transform: scale(1);
   }
 }
 
 @keyframes diffZoomIn3 {
-  0%{
+  0% {
     transform: scale(0);
   }
-  100%{
+  100% {
     transform: scale(1);
   }
 }