Stop inserting space in LESS variable property access

prettier · Jan 3, 2023 · 70f97d3 · 70f97d3
1 parent 41cee06
commit 70f97d3
Show file tree

Hide file tree

Showing 3 changed files with 37 additions and 6 deletions.
diff --git a/changelog_unreleased/less/14103.md b/changelog_unreleased/less/14103.md
@@ -0,0 +1,17 @@
+#### Do not insert space in LESS property access (#14034 by @mvorisek)
+
+<!-- prettier-ignore -->
+```less
+// Input
+a {
+  color: @colors[@white];
+}
+
+// Prettier stable
+a {
+  color: @colors[ @white];
+}
+
+// Prettier main
+<Same as input>
+```
diff --git a/src/language-css/printer-postcss.js b/src/language-css/printer-postcss.js
@@ -620,7 +620,21 @@ function genericPrint(path, options, print) {
         }
 
         // Ignore `@` in Less (i.e. `@@var;`)
-        if (iNode.type === "value-atword" && iNode.value === "") {
+        if (
+          iNode.type === "value-atword" &&
+          (iNode.value === "" ||
+            /*
+            @var[ @notVarNested ][notVar]
+            ^^^^
+            */
+            iNode.value.endsWith("[") ||
+            /*
+            @var[ @notVarNested ][notVar]
+                                ^^^^^^^^^
+            */
+            (iNextNode?.type === "value-word" &&
+              iNextNode.value.startsWith("]")))
+        ) {
           continue;
         }
 

diff --git a/tests/format/less/case/__snapshots__/jsfmt.spec.js.snap b/tests/format/less/case/__snapshots__/jsfmt.spec.js.snap
@@ -453,11 +453,11 @@ printWidth: 80
 };
 
 @not-var {
-  canchangecase: @var[ @notVarNested][notVar];
-  canchangecase: @var[ @notVarNested][notVar];
-  canchangecase: @var[ @notVarNested ][notVar];
-  canchangecase: @var[ @notVarNested][ notVar];
-  canchangecase: @var[ @notVarNested][notVar ];
+  canchangecase: @var[@notVarNested][notVar];
+  canchangecase: @var[@notVarNested][notVar];
+  canchangecase: @var[@notVarNested][notVar];
+  canchangecase: @var[@notVarNested][notVar];
+  canchangecase: @var[@notVarNested][notVar];
   canchangecase: @var[notVar];
 }