Potential autofix improvement for jsx-wrap-mulitlines

[sharmilajesupaul]

This change removes the newline that comes before opening parentheses, if "parens-new-line" is specified as an option and the multiline JSX starts on a new line.

i.e. when "parens-new-line" is specified, instead of fixing these cases:

  <div prop={
    <div>
      <p>Hello</p>
    </div>
  }>
    <p>Hello</p>
  </div>

// &

  <div>
    {foo &&
      <div>
        <p>Hello World</p>
      </div>
    }
  </div>

like this:

  <div prop={
(
    <div>
      <p>Hello</p>
    </div>
)
  }>

// &

  <div>
    {foo &&
(
      <div>
        <p>Hello World</p>
      </div>
)
}
  </div>

this change will inline the opening parentheses:

  <div prop={(
    <div>
      <p>Hello</p>
    </div>
)}>

// &

  <div>
    {foo && (
      <div>
        <p>Hello World</p>
      </div>
)}
  </div>

I did not change the position of closing parens in this PR, I'm not sure if there is another rule that might cover that? if not, I can also try and collapse the bottom parens to be more consistent.

cc. @ljharb

Update: this change will now inject the closing paren before the token after the jsx body, so the autofix will not leave the closing paren on its own line.

[ljharb]

This LGTM but I think it does need to cover the closing paren also

[ljharb]

i think this one needs to be )}> on the next line

[sharmilajesupaul]

@ljharb added change to cover closing paren, ptal thanks!