Doesn't break long React props

[erbridge]

e.g.

<BaseForm url="/auth/google" method="GET" colour="blue" size="large" submitLabel="Sign in with Google" />

should probably be converted into

<BaseForm 
  url="/auth/google"
  method="GET"
  colour="blue"
  size="large"
  submitLabel="Sign in with Google"
/>

[wmertens]

And don't forget about this form:

const FooForm = ({colour}) => {
   const method = 'GET'
   return (
      <BaseForm {...{
        url: '/auth/google',
        method, colour
      }}/>
   )
}

[ericclemmons]

Similarly, I tested:

const render = ({ styles }) => (
  <div style={styles} key="something">
    Keep the wrapping params. Put each key on it's own line.
  </div>
);

which results in:

const render = ({ styles }) => <div style={
  styles
} key="something">
  Keep the wrapping params. Put each key on it's own line.
</div>;

[gaearon]

Here, IMO the best style would be the one that optimizes for diff readability on changes.
Like the one in the OP post.
Probably worth doing for any elements that don't fit within a single line completely.

[rattrayalex]

I'm going to try to tackle this now.

[rattrayalex]

The example by @ericclemmons now compiles to:

const render = ({ styles }) => <div style={styles} key="something">
  Keep the wrapping params. Put each key on it's own line.
</div>;

which is an improvement, but I agree the parens should be maintained. I'll open a separate issue for that if one doesn't already exist, and try to tackle soon.

(note that improvement is probably unrelated to my modification, I didn't check)

[vramana]

I am closing this in favour of #73. Please continue the discussion in that thread.