Fix prettier-ignore inside JSX

changelog_unreleased/javascript/pr-7877.md

@@ -0,0 +1,29 @@
+#### Fix `prettier-ignore` inside JSX ([#7877](https://github.com/prettier/prettier/pull/7877) by [@fisker](https://github.com/fisker))
+
+<!-- prettier-ignore -->
+```jsx
+// Input
+<div>
+{
+  /* prettier-ignore */
+  x     ?   <Y/> : <Z/>
+}
+</div>;
+
+// Prettier stable
+<div>
+  {/* prettier-ignore */
+  x     ?   <Y/> : <Z/>}
+</div>;
+
+// Prettier stable (Second format)
+<div>{/* prettier-ignore */ x     ?   <Y/> : <Z/>}</div>;
+
+// Prettier master
+<div>
+  {
+    /* prettier-ignore */
+    x     ?   <Y/> : <Z/>
+  }
+</div>;
+```

src/language-js/printer-estree.js

@@ -2170,26 +2170,24 @@ function printPathNoParens(path, options, print, args) {
     case "JSXExpressionContainer": {
       const parent = path.getParentNode(0);
 
-      const preventInline =
-        parent.type === "JSXAttribute" &&
-        n.expression.comments &&
-        n.expression.comments.length > 0;
+      const hasComments =
+        n.expression.comments && n.expression.comments.length > 0;
 
       const shouldInline =
-        !preventInline &&
-        (n.expression.type === "ArrayExpression" ||
-          n.expression.type === "ObjectExpression" ||
-          n.expression.type === "ArrowFunctionExpression" ||
-          n.expression.type === "CallExpression" ||
-          n.expression.type === "OptionalCallExpression" ||
-          n.expression.type === "FunctionExpression" ||
-          n.expression.type === "JSXEmptyExpression" ||
-          n.expression.type === "TemplateLiteral" ||
-          n.expression.type === "TaggedTemplateExpression" ||
-          n.expression.type === "DoExpression" ||
-          (isJSXNode(parent) &&
-            (n.expression.type === "ConditionalExpression" ||
-              isBinaryish(n.expression))));
+        n.expression.type === "JSXEmptyExpression" ||
+        (!hasComments &&
+          (n.expression.type === "ArrayExpression" ||
+            n.expression.type === "ObjectExpression" ||
+            n.expression.type === "ArrowFunctionExpression" ||
+            n.expression.type === "CallExpression" ||
+            n.expression.type === "OptionalCallExpression" ||
+            n.expression.type === "FunctionExpression" ||
+            n.expression.type === "TemplateLiteral" ||
+            n.expression.type === "TaggedTemplateExpression" ||
+            n.expression.type === "DoExpression" ||
+            (isJSXNode(parent) &&
+              (n.expression.type === "ConditionalExpression" ||
+                isBinaryish(n.expression)))));
 
       if (shouldInline) {
         return group(

tests/jsx_ignore/__snapshots__/jsfmt.spec.js.snap

@@ -91,14 +91,18 @@ f(
 
 // this should remain as-is
 <div>
-  {/* prettier-ignore */
-  foo ( )}
+  {
+    /* prettier-ignore */
+    foo ( )
+  }
 </div>;
 
 // this should remain as-is
 <div>
-  {/* prettier-ignore */
-  x     ?   <Y/> : <Z/>}
+  {
+    /* prettier-ignore */
+    x     ?   <Y/> : <Z/>
+  }
 </div>;
 
 push(

tests/line_suffix_boundary/__snapshots__/jsfmt.spec.js.snap

@@ -56,7 +56,8 @@ ExampleStory.getFragment('story')}
 \`;
 
 <div>
-  {ExampleStory.getFragment("story") // $FlowFixMe found when converting React.createClass to ES6
+  {
+    ExampleStory.getFragment("story") // $FlowFixMe found when converting React.createClass to ES6
   }
 </div>;
 

tests_config/run_spec.js

@@ -35,7 +35,6 @@ const unstableTests = new Map(
       (options) => options.printWidth === 1,
     ],
     "js_empty/semicolon.js",
-    "jsx_ignore/jsx_ignore.js",
     "markdown_footnoteDefinition/multiline.md",
     "markdown_spec/example-234.md",
     "markdown_spec/example-235.md",