Fixed missing commas in "font-family" property

ThisIsManta · Apr 20, 2017 · 28b2e9e · 28b2e9e
1 parent 3b4daef
commit 28b2e9e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/edge/format.js b/edge/format.js
@@ -297,7 +297,11 @@ function format(content, options) {
 				// Insert the property value(s) without the last portion of comments
 				// For example,
 				// margin: 8px 0; /* right-comment */
-				outputBuffer.append(propertyValues.join(' '))
+				if (propertyName === 'font-family') {
+					outputBuffer.append(propertyValues.join(comma))
+				} else {
+					outputBuffer.append(propertyValues.join(' '))
+				}
 
 				// Put the last portion of comments aside
 				if (commentsOnTheRight.length > 0) {

diff --git a/spec/font-face/input.styl b/spec/font-face/input.styl
@@ -1,2 +1,5 @@
 @font-face
-  font-family Gensans
+  font-family Gensans
+
+body
+  font-family 'graphik', Arial, Helvetica, sans-serif
diff --git a/spec/font-face/output.styl b/spec/font-face/output.styl
@@ -1,3 +1,7 @@
 @font-face {
 	font-family: Gensans;
+}
+
+body {
+	font-family: 'graphik', Arial, Helvetica, sans-serif;
 }