Support comma first style of variable declaration

Allow code to be formatted like this:

var a = 1
  , b = "somethign else"
  , isAwesome = true;

And apparently, we have a somewhat ready patch available. Would be great if this can get included in here!

[tgirardi]

I’ve been working on a patch to support this feature “on demand” (the user can enable or disable the option of using comma first with a setting I’ve created).

I’ve achieved the following formatting for variable declaration (using 2 spaces):

var firstVar = 1
  , secondVar = 2
  , thirdVar = 3
;

But I have some doubts.

How should we treat arrays, objects and arguments declarations? Lately I’ve been using the following format:

myArray = [
  item1
, item2
, item3
];

myObject = {
  prop1: 1
, prop2: 2
, prop3: 3
}

Which, as you can see, is not exactly the same as the variable declaration formatting: this last one includes a +1 indentation level for the second variable (notice the two spaces before the comma that are not present before the comma for “item2” nor for the one for “prop2”). Var declarations use this extra indentation in order to align in the same column the start of each variable name as stated by @lukemartin.

The reasons for using the formatting shown on the above code are:

1.- To avoid the linting errors that would be thrown by jshint (indentation errors). For example:

myArray = [
    item1
  , item2
];

Throws “Expected ] to have an indentation at 3, instead at 1”. If we follow the suggestion, we get a very ugly result.

2.- To maintain the advantage of aligning the start of each property name, array item, etc. For example:

myArray = [
  item1
  , item2
];

Also doesn’t produce a linting error, but doesn’t look as nice as the examples above.

With a more clear idea of how I should treat those cases I could finish the implementation of this feature and issue a Pull Request.

[evocateur]

What about 4 spaces per tab? Actual tabs?

On Fri, Nov 8, 2013 at 9:36 AM, Luke Martin [e-mail] wrote:

+1 for comma-first support. At 2-space indentation, it’s the tidiest way of arranging variables var foo = true , bar = ‘hello’

, something;

Reply to this email directly or view it on GitHub: https://github.com/einars/js-beautify/issues/245#issuecomment-28081629