New Rule Proposal for jsx-curly-spacing

[Dindaleon]

I would like to be able to do the following:

"jsx-curly-spacing": [2, "always", {"alternative": true}]

When "always" is used and alternative is true, following patterns are not warnings:

<Hello name={ firstname } />;
<Hello name={{ firstname: 'John', lastname: 'Doe' }} />;
<Hello name={
  firstname
} />;

the following patterns are considered warnings:

<Hello name={firstname} />;
<Hello name={ firstname} />;
<Hello name={firstname } />;
<Hello name={ {firstname: 'John', lastname: 'Doe'} } />;

[yannickcr]

If I understand correctly you would like something similar to the objectsInObjects option in the ESLint object-curly-spacing rule. I think that's something we can do.

[Dindaleon]

Awesome! Thank you.

[lencioni]

I think this is similar to #211.

[casesandberg]

I get Configuration for rule "react/jsx-curly-spacing" is invalid: Value "data["1"].alternative" has additional properties when trying to do "alternative": true as a third argument

[yannickcr]

@casesandberg yes, alternative is not a valid option since this proposal is not implemented yet.

[casesandberg]

Ah, got it @yannickcr, I will see if I can get it in tonight!