New Rule Proposal: jsx-indent

[tuures]

Like the indent rule from the core but for jsx lines too as right now they are being ignored.

The following would be considered as an error:

/*eslint jsx-indent: [2, 2]*/
Foo(a) => (
  <div>
<span>{a /* wrong indent here*/}</span>
  </div>
);
foo();

The following would not be considered as an error:

/*eslint jsx-indent: [2, 2]*/
Foo(a) => (
  <div>
    <span>{a}</span>
  </div>
);
foo();

[LinusU]

I would love to see this!, I think it should work just as the core indentation does, but for JSX lines. Hopefully it shouldn't be two hard to implement.

[BerkeleyTrue]

👍

[DanielSundberg]

Changelog says "Add jsx-intent rule", so the rule seems to be named "jsx-intent"? Here you talk about "jsx-indent"?

I'm not native in english but intent is something else than indent in my vocabulary...

The eslint rule for JS is called "indent".

[DanielSundberg]

Ok, I can see now that the rule is actually called "jsx-indent", perhaps just a typo in the changelog then...

[yannickcr]

Yeah, it is a typo in the changelog :|