Merge pull request #10802 from internetarchive/prettier

Use the prettier plugin for stylelint to format css/less files

Author: cdrini

.pre-commit-config.yaml

@@ -92,9 +92,10 @@ repos:
     - id: stylelint
       files: \.(css|less)$
       additional_dependencies:
-        - stylelint@15.10.1
-        - stylelint-declaration-strict-value@1.9.2
-        - postcss-less@3.1.4
+        - stylelint@16.19.1
+        - stylelint-declaration-strict-value@1.10.11
+        - postcss-less@6.0.0
+        - stylelint-prettier@5.0.3
       args: [--fix]
 
   - repo: local

.stylelintrc.json

@@ -1,4 +1,5 @@
 {
+  "extends": ["stylelint-prettier/recommended"],
   "customSyntax": "postcss-less",
   "plugins": [
     "stylelint-declaration-strict-value"
@@ -24,14 +25,8 @@
 		"declaration-block-no-redundant-longhand-properties": true,
 		"no-duplicate-selectors": true,
 		"block-no-empty": true,
-		"indentation": 2,
 		"function-url-quotes": "never",
 		"length-zero-no-unit": true,
-		"max-empty-lines": 1,
-		"no-eol-whitespace": true,
-		"number-leading-zero": "never",
-		"no-missing-end-of-source-newline": true,
-		"max-line-length": 100,
 		"shorthand-property-no-redundant-values":true,
 		"selector-max-specificity": "1,4,0",
 		"unit-no-unknown": true,
@@ -44,13 +39,9 @@
 		"at-rule-no-unknown": true,
 		"comment-no-empty": true,
 		"no-duplicate-at-import-rules": true,
-		"no-extra-semicolons": true,
 		"no-empty-source": true,
 		"font-family-no-duplicate-names": true,
 		"font-family-no-missing-generic-family-keyword": [true, {"ignoreFontFamilies": ["!important"]}],
-		"no-descending-specificity": true,
-		"string-quotes": ["double", {"avoidEscape":false}],
-		"declaration-bang-space-before": "always",
-		"declaration-colon-space-after": "always"
+		"no-descending-specificity": true
   }
 }

openlibrary/plugins/openlibrary/js/ile/utils/SelectionManager/SelectionManager.less

@@ -9,8 +9,8 @@
   box-shadow: 0 0 10px 2px inset @selection-outline;
 }
 .ile-selectable {
-  box-shadow: 0 0 0 2px inset rgba(0,0,0,.1);
-  transition-duration: .2s;
+  box-shadow: 0 0 0 2px inset rgba(0, 0, 0, 0.1);
+  transition-duration: 0.2s;
   transition-property: background-color, box-shadow;
   border-radius: 4px;
 }
@@ -28,15 +28,15 @@
   background: @ile-toolbar-background;
   color: @ile-toolbar-text-color;
   z-index: @ile-toolbar-z-index;
-  font-size: .9em;
+  font-size: 0.9em;
 }
 #ile-toolbar a {
   padding: 4px 16px;
   display: inline-block;
   color: @ile-toolbar-text-color;
   text-decoration: none;
   border-left: 2px dotted @ile-toolbar-text-color;
-  transition: background .2s;
+  transition: background 0.2s;
   cursor: pointer;
 }
 #ile-selection-actions {
@@ -73,7 +73,9 @@
 #ile-drag-status .images li:first-child {
   flex-shrink: 0;
 }
-#ile-drag-actions a:hover { background: rgba(0,0,0,.15); }
+#ile-drag-actions a:hover {
+  background: rgba(0, 0, 0, 0.15);
+}
 
 a.ile-selectable[href^="/authors/OL"] {
   padding-right: 4px;
@@ -83,9 +85,9 @@ a.ile-selectable[href^="/authors/OL"] {
 .ile-select-handle {
   padding: 0 5px;
   text-decoration: none;
-  opacity: .2;
+  opacity: 0.2;
   cursor: default;
-  transition: opacity .2s;
+  transition: opacity 0.2s;
   color: @selection-outline;
 }