Merge pull request #2973 from karl/jsx-collapse-multiple-jsx-whitespaces

Collapse multiple JSX whitespaces

src/printer.js

@@ -4148,11 +4148,16 @@ function printJSXElement(path, options, print) {
       children[i] === jsxWhitespace &&
       children[i + 1] === "" &&
       (children[i + 2] === softline || children[i + 2] === hardline);
+    const isDoubleJSXWhitespace =
+      children[i] === jsxWhitespace &&
+      children[i + 1] === "" &&
+      children[i + 2] === jsxWhitespace;
 
     if (
       (isPairOfHardlines && containsText) ||
       isPairOfEmptyStrings ||
-      isLineFollowedByJSXWhitespace
+      isLineFollowedByJSXWhitespace ||
+      isDoubleJSXWhitespace
     ) {
       children.splice(i, 2);
     } else if (isJSXWhitespaceFollowedByLine) {

tests/jsx-text-wrap/__snapshots__/jsfmt.spec.js.snap

@@ -298,7 +298,16 @@ x =
     </div>
   </div>
 
+x =
+  <div>
+    {" "} <div>text</div>
+  </div>
 
+// NOTE: Multiple JSX whitespaces are collapsed into a single space.
+x =
+  <div>
+    {" "}{" "}{" "}
+  </div>
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 // Wrapping text
 x = (
@@ -679,4 +688,14 @@ x = (
   </div>
 );
 
+x = (
+  <div>
+    {" "}
+    <div>text</div>
+  </div>
+);
+
+// NOTE: Multiple JSX whitespaces are collapsed into a single space.
+x = <div> </div>;
+
 `;

tests/jsx-text-wrap/test.js

@@ -295,4 +295,13 @@ x =
     </div>
   </div>
 
+x =
+  <div>
+    {" "} <div>text</div>
+  </div>
 
+// NOTE: Multiple JSX whitespaces are collapsed into a single space.
+x =
+  <div>
+    {" "}{" "}{" "}
+  </div>