wrap_line_length breaking in confusing places

[jackocnr]

It breaks after negation e.g.

if ( .... lots of conditions here ... && !someVar) {

becomes

if ( .... lots of conditions here ... && !
    someVar) {

and also before property names e.g.

if ( .... lots of conditions here ... && someVar.someProperty) {

becomes

if ( .... lots of conditions here ... && someVar
    .someProperty) {

Both make the code way less readable. Perhaps it should only break on spaces? Or between logical operators e.g. && or ||?

[jdavisclark]

ultimately this is going to be an einars/js-beautify issue. I'll open an issue up there and reference this one.

[webbushka]

Having a similar issue with ES6 modules something like this

var App = function(){};

export default App;

turns into:

var App = function(){};

export
default App;

is this also a js-beautify issue?

[jdavisclark]

@webbushka yes, but this is a separate issue. Looks like beautifier/js-beautify#382 is already open to address this.