Add support for conditional rendering to jsx-multiline-wrap

[jseminck]

Main issue: #1282

I tried to make the fixer better so the output has the parens in the correct place. Probably, need to add a few more tests, but I think the current solution looks quite solid.

The rule is defaulted to false initially. If enabling it by default, a bigger version bump would be required?

See also some comments on another issue regarding jsx-multiline-wrap: #1207 (comment)

[jseminck]

FYI broken build doesn't seem to be related to this branch. Currently when I try to build master I get the same error messages, even though the latest build on travis for master was successful (21hrs ago: https://travis-ci.org/yannickcr/eslint-plugin-react/builds/250125668)

[deecewan]

do you mean {conditional: true}?

[deecewan]

I know that it'd require a bigger version bump, but to me it'd make more sense to have all the rules enabled by default.

Also, while you're in there, feel like adding ternaries?

[ljharb]

Enabling rules by default should wait for a separate semver-major PR, released after the non-major changes are released.

[marcdavi-es]

*** sits wriggling with excitement waiting for the merge ***

[ljharb]

Sorry for the inconvenience; I've just merged a refactor to start using arrow functions. Would you mind rebasing?

[jseminck]

Are there some problems regarding for this PR to be moved forward?

I'd like to take a further look at this rule because I currently joined a project where we use a lot of conditional rendering and eslint-plugin-react doesn't always fix things as I expect it to. 😄

[jseminck]

Seems like this was added in another PR, hence closing.

To achieve this functionality, use the option: {logical: 'parens'} or {logical: 'parens-new-line'}.

[ljharb]

@jseminck sorry about the conflict :-)