The first JSX property should always be placed on a new line if the JSX tag takes up multiple lines and there are multiple properties (react/jsx-first-prop-new-line)

[feross]

Ensure correct position of the first property.

multiline-multiprop: The first property should always be placed on a new line if the JSX tag takes up multiple lines and there are multiple properties.

https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-first-prop-new-line.md

I've personally run into this while working on WebTorrent Desktop and Brave with @dcposch, where we've been very inconsistent with this.

[dcousens]

IMO

OK:

// single-line
<Hello personal={true} />

// follows single-line, but with object spacing
<Hello foo={{
  x: 1,
  ...
}} />

// still single-line with object spacing
<Hello bar foo={{
  x: 1,
  ...
}} personal={true} />

// too many props on 1 line,  therefore, like an object definition, go multi-line
<Hello
    personal={true}
    foo="bar"
/>

Bad:

<Hello
    personal />

<Hello personal
    prop />

<Hello personal={true}
    foo="bar"
/>

(so multiline-multiprop?)

[feross]

Yeah, that looks right @dcousens!

[feross]

Just ran the test suite. No errors, except on webtorrent-desktop, and I agree that all the things it caught should be changed. So this LGTM

[feross]

This will ship in standard 14.