dot chained calls are not indented correctly

[nisaacson]

When chaining calls together via the . operator, the following lines are not indented correctly

Desired

one()
  .then(two)
  .then(function() {
  })
  .fail(failHandler)

Current

one()
.then(two)
.then(function() {
})
.fail(failHandler)

Is there a way I can add a custom regular expression to perform additional indentation after the standard vim-javascript indent plugin has finished? Adding an additional two spaces to any line that begins with . would be a vast improvement

[qstrahl]

Is there a way I can add a custom regular expression to perform additional indentation after the standard vim-javascript indent plugin has finished?

Don't bother; I think this (indenting the chained calls) should be the default behaviour, we'll likely add support if the other maintainers agree.

[amadeus]

I would love this functionality as well, unfortunately I haven't been able to spend the time to figure out how the indent files actually work, so I am unable to fix at the this time.

[slindberg]

👍 I would have done this already if simply adding a regex would work... alas the indentation logic is quite complicated.

[goatslacker]

yes it is. Although I can probably get this in, it should be similar to the comma logic.

[cpixl]

A great feature that is missing! +1

[FrigoEU]

Any news on this one?

[jeffrom]

+1 would love this

[danielchabr]

I would also greatly appreciate this feature +1

[smdahlen]

+1

[doxavore]

I'm not a vim scripter, but I have some very basic support for this here: https://github.com/doxavore/vim-javascript/tree/dot-chained-indent

It handles things like

one()
  .then(two)
  .then(function() {
  })

But stumbles a little with the original example, creating:

one()
  .then(two)
  .then(function() {
  })
    .fail(failHandler)

If someone has some time to work on this, perhaps it will offer a start.

[bryankennedy]

Nice work doxavore. That is partially working for me. If I visually select a group it will correctly format things with an = call.

However, it doesn't help if I'm writing a statement like this:

var thing = one.two()
    .chain()

and then hit enter and type, I end up with this.

var thing = one.two()
    .chain()
.nextChain()

I'm also not really a Vim Scripter and not sure how plugins handle this sort of eventuality.

[anshul]

@qstrahl @amadeus Any updates?

[jameslai]

+1

[qstrahl]

I really need to get around to learning how the indentation stuff works, because the absence of this feature has been bothering me for months. I'll try to look into it tonight. We'll see if I can't shake a stick at implementing decent behaviour for this.

[amadeus]

@qstrahl I've poked around a bit in the file, and it's not gonna be an easy task.

As for everyone else in this thread, the person who wrote the indent logic for this originally is no longer contributing to the project. So editing/updating it is going to require one or some of us to figure it out. And it's been tough to find time to do this. Please note we aren't just ignoring the issue, I would love this feature too.