jsx-curly-brace-presence warns incorrectly on trailing whitespace

[taion]

This is a regression from #2409, cc @vedadeepta @ljharb

Per #2409 (comment), when running autofix with this change, I now get e.g.

           <div>
-            Confirm{' '}
+            Confirm
             <button type="button" onClick={this.onClickYes}>

The rule may be correct in isolation, but this interacts very poorly with the standard automatic removal of trailing whitespace.

[vedadeepta]

@taion you want to keep the traling white space iff it is inside curly braces ?

I think the rule can be adjusted for this special case but i'm not sure if markup should be written like this though.

[taion]

jsx-curly-brace-presence should not warn when the curly braces surround trailing whitespace.

How else would you write this markup? In general it's not that odd to put a line break in this sort of place. It's e.g. what Prettier does.

[ljharb]

That also makes total sense.

[ljharb]

@vedadeepta indeed, putting whitespace inside curlies is the only way to write it so that HTML doesn't collapse it.

[vedadeepta]

@taion makes sense to account for your case, earlier i was thinking more in terms of css paddings/margins for creating spaces.

[ryansully]

I think I'm experiencing a related situation where I need whitespace between children, but the whitespace child warns incorrectly.

<button>
  <i className="fa fa-check" />
  {' '}
  Confirm
</button>

In this case I'm also using jsx-one-expression-per-line, so it's not just an issue of trailing whitespace, but of any use of whitespace children.

There is also the case if there needs to be non-whitespace text with leading or trailing whitespace, or both:

<p>
  {'The '}
  <strong>rest</strong>
  {' of the '}
  <a href="/">site</a>
  {' already.'}
</p>

An autofix of this results in joined words with only one space in the middle.