move-line-up/down command breaks JSX indentation

[jgebhardt]

Repro steps

Using the following test file:

<div>
  <div>
    <p>
      first line
    </p>
  </div>
  <div>second line</div>
</div>

Select the first child of the enclosing <div />:

Use the editor:move-line-down / editor:move-line-up (cmd+ctrl+up/down) command to move the selection around. This breaks the indentation by setting each line to the first line's indent level.

[gandm]

OK Thanks. I'll take a look. In the meantime just cursor into the last row moved and it will re-indent the previous rows.

[gandm]

@jgebhardt Looks like I'm indenting it correctly but on return Atom re-indents at https://github.com/atom/atom/blob/master/src/language-mode.coffee#L235 using the previous line's indent.

I may have to monkey patch this method which I'd hoped to avoid.

[gandm]

Incorrect scopes returned by atom.

[gandm]

@jgebhardt Should now be fixed in 2.15.6 release.