Long names cause strange line-breaks with Arrow Functions about prettier

[jlongster]

@rattrayalex Sure! If I didn't just solve it, I'll leave it up to you :) I think I may have something works.

[jlongster]

Ok, let's try this. I'm curious if this causes any other awkward formatting, but I think this works.

[jlongster]

I think you should just move it into a proper block:

var oneLongVariableName = (foo, bar) => {
  return this.longerMethodName(c, service).forEach(otherLongFunctionName);
}

We don't really support arrow functions without blocks with code on the next line. It would be a hard special-case to format correctly without breaking other formats that involve arrow expressions.

[RamIdeas]

(Watching the twitch live stream)

I possible solution that would respect the case above, and the chains of - for example - promises, could be to align on the period, like:

var variable = (foo, bar) => this.methodName(...)
                                 .forEach(...);

promise.then( () => ... )
       .then( () => ... )
       .then( () => ... )

[rattrayalex]

I would really love to see this re-opened... cool if I try to tackle it as a first contribution? 😉

(EDIT: he is working on it again)