jsx-indent and indent require the same indentation amount

[Wilfred]

Related to #898, but I think it's sufficiently different to warrant a separate issue.

Given the JS:

import React from 'react';

function foo() {
  if (true) {
    if (true) {
      if (true) {
        someText = [
          <span>{text}</span>,
          otherText,
        ];
      }
    }
  }
}

and following eslint configuration:

{
    "parserOptions": {
        "ecmaVersion": 6,
        "ecmaFeatures": {
            "jsx": true,
            "impliedStrict": true
        },
        "sourceType": "module"
    },
    "rules": {
        "indent": ["warn", 2],
        "react/jsx-indent": "warn"
    },
    "plugins": [
        "react"
    ]
}

Then I get a warning Expected indentation of 12 space characters but found 10. for the <span>.

I suspect this is because the default indent for jsx-indent is 4 spaces, whereas I've configured indent to use 2 spaces.

I think that jsx-indent should either be able to handle different indent values to indent, or perhaps it should just reuse the existing indent value?

[ljharb]

Rules can't access other rules' config. If you want 2 spaces, you have to configure both rules.

[tswaters]

This raises the question as to why there are two rules.

Could it not work similar how the eslint indent rule works (allowing override on SwitchCase, among others, if desired) -- allow it to accept extra parameters to overwrite specific things? e.g.,

"react/jsx-indent": ["error", 2, {props: 4}]

By default, it could use the same value.

[ljharb]

@tswaters there are two rules because JSX isn't part of JS, so eslint core doesn't tend to have rules that can work with it.

[tswaters]

My mistake, I misread the original post.

I was talking about react/jsx-indent and react/jsx-indent-props being separate.... it seems to me these should be the same with -props controlled via options similar to how the core indent rule works in eslint.

[ljharb]

That does seem reasonable - however, people indent their props in far crazier ways then they indent their object literal keys, so it might get unwieldy.

The first step would be enhancing jsx-indent to handle all the props cases, and ship that as a semver-minor - and then later, deprecate/remove jsx-indent-props.

@yannickcr, thoughts?

[blissdev]

I've encountered the same issue where after the ESLint 4 upgrade, the JSX and ESLint indent levels have to match up otherwise there are conflicts.

[HHogg]

After also doing the upgrade from

eslint@^3.7.1
eslint-plugin-react@^6.0.0

... to

eslint@^4.4.0
eslint-plugin-react@^7.1.0

The indent and jsx-indent conflict with one another... I've just disabled indent for the moment.

[ljharb]

@HHogg you can instead configure indent to ignore JSX and delegate to eslint-plugin-react, which solves the problem.

[fruitraccoon]

@ljharb Any chance you have an example of how to setup the indent rule to ignore JSX?

I have the same situation as @HHogg. Looking at the docs for indent, I'm guessing that the ignoredNodes option is the way to do this, but eslint selectors are a bit outside my skill set at the moment and a working example to start from would be great! :)

[ljharb]

https://eslint.org/docs/rules/indent#ignorednodes, with a selector of "JSX*" should do it.

[fruitraccoon]

Thank very much for the quick reply @ljharb - using your suggestion I was able to get something working. Using the wildcard character on the syntax tree node name didn't work for me, but by using the rule below, I seem to have about the equivalent of how our linting used to work:

"indent" : ["error", 4, { "ignoredNodes": ["JSXElement *"] }],

I believe that this will just turn off the indent rule for anything to do with JSX, including within any JSX expressions.

A side note for anyone else exploring this - I found this site very useful for finding out what the AST node names actually are (https://astexplorer.net/).

[ljharb]

@Wilfred does this solve your issue?

[Wilfred]

I ended up configuring both plugins to just use the same indent, so I'm happy.

In a perfect world the plugins would detect the conflicting settings, but I don't see how that's possible. It's just a small papercut.

I'm happy to close this if you don't think there's anything else we can do here?

[ljharb]

Unless eslint core agrees to make a global "indent" setting that the core indent rule reads (and then that jsx-indent could read), there's nothing we can do.