Weird Indentation with chained calls

Prettier 1.15.3 Playground link

--parser babylon

Input:

$.ajax({
    url: 'https://random.com'
}).done(function (data) {
    console.log('Done');
}).fail(function () {
    console.log('Error');
});

Output:

$.ajax({
  url: "https://random.com"
})
  .done(function(data) {
    console.log("Done");
  })
  .fail(function() {
    console.log("Error");
  });

Expected behavior:

$.ajax({
    url: 'https://random.com'
}).done(function (data) {
    console.log('Done');
}).fail(function () {
    console.log('Error');
});

If I remove the fail from the chained call list then prettier will format it like this -

Prettier 1.15.3 Playground link

--parser babylon

Input:

$.ajax({
    url: 'https://random.com'
}).done(function (data) {
    console.log('Done');
});

Output:

$.ajax({
  url: "https://random.com"
}).done(function(data) {
  console.log("Done");
});

[duailibe]

@bodrick Because with 2+ chains you’re most likely to want something similar to:

a
  .step(() => {
    //
  })
  .step(() => {
    //
  })
  .step(() => {
    //
  });

which is subjectively better than:

a.step(() => {
  //
}).step(() => {
  //
}).step(() => {
  //
});

We had to choose an heuristic and that was it.

As for the OP, I personally prefer the Prettier formatting than the your expected.

[pstanton]

agree to disagree, but for me double indentation is better than inconsistent indentation.

ie you have double indentation for all of the other functions…