Fixed removing colons causing Stylus compilation failures

See #79
ThisIsManta · Jan 31, 2021 · 0734bec · 0734bec · qnp · Jan 31, 2021
1 parent 258de26
commit 0734bec
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/edge/format.js b/edge/format.js
@@ -379,6 +379,9 @@ function format(content, options = {}) {
 					}
 				}
 
+				// See https://github.com/ThisIsManta/stylus-supremacy/issues/79
+				const preserveColons = nodesExcludingCommentsOnTheRight.length === 1 && nodesExcludingCommentsOnTheRight[0] instanceof Stylus.nodes.Expression
+
 				// Insert the property value(s) without the last portion of comments
 				if (nodesExcludingCommentsOnTheRight.every(node => node instanceof Stylus.nodes.Expression)) {
 					const numberOfLineTaken = _.chain(nodesExcludingCommentsOnTheRight)
@@ -393,15 +396,15 @@ function format(content, options = {}) {
 						outputBuffer.append(propertyValues.join((inputNode.expr.isList ? ',' : '') + options.newLineChar + innerIndent))
 
 					} else {
-						if (options.insertColons) {
+						if (options.insertColons || preserveColons) {
 							outputBuffer.append(':')
 						}
 						outputBuffer.append(' ')
 						outputBuffer.append(propertyValues.join(inputNode.expr.isList ? comma : ' '))
 					}
 
 				} else {
-					if (options.insertColons) {
+					if (options.insertColons || preserveColons) {
 						outputBuffer.append(':')
 					}
 					outputBuffer.append(' ')

diff --git a/spec/option-insert-colon-false/input.styl b/spec/option-insert-colon-false/input.styl
@@ -2,4 +2,8 @@ body
   display none
 
 @media (min-width: 100px)
-  display none
+  display none
+
+.foo
+  color: $colors.red $colors.blue 3
+  display block
diff --git a/spec/option-insert-colon-false/output.styl b/spec/option-insert-colon-false/output.styl
@@ -4,4 +4,9 @@ body {
 
 @media (min-width 100px) {
 	display none;
+}
+
+.foo {
+	color: $colors.red $colors.blue 3;
+	display block;
 }