Add support for Computed / Dynamic Object Property Names

Description

When formatting an object literal with a computed property, an undesired new line is introduced after the object’s initial curly brace.

This issue was originally filed in the vscode extension’s repo at this url: https://github.com/HookyQR/VSCodeBeautify/issues/143

Input

The code looked like this before beautification:

export default {
    filesById({ files }, getters) {
        return files.reduce((memo, f) => Object.assign(memo, { [f.id]: f }), {})
    },
}

Expected Output

The code should not have changed after beautification:

export default {
    filesById({ files }, getters) {
        return files.reduce((memo, f) => Object.assign(memo, { [f.id]: f }), {})
    },
}

Actual Output

The code actually looked like this after beautification:

export default {
    filesById({ files }, getters) {
        return files.reduce((memo, f) => Object.assign(memo, {
            [f.id]: f }), {})
    },
}

Steps to Reproduce

Environment

• Operating System: OS X 10.11.6
• VS Code Version: 1.12.2
• beautify Version: 1.0.2 (vscode extension)

Settings

Example:

{
    "end_with_newline": true,
    "indent_char": " ",
    "indent_size": 4,
    "indent_with_tabs": false,
    "max_preserve_newlines": 2,
    "wrap_line_length": 120,
    "indent_body_inner_html": false,
    "indent_scripts": "normal",
    "wrap_attributes": "force-aligned",
    "newline_between_rules": false,
    "break_chained_methods": false,
    "comma_first": false,
    "keep_array_indentation": false,
    "keep_function_indentation": false,
    "operator_position": "after-newline",
    "space_before_conditional": true,
    "space_in_paren": false,
    "space_in_empty_paren": false,
    "js": {
        "brace_style": "collapse,preserve-inline"
    }
}

[bitwiseman]

@alexsasharegan - thanks for taking the time, that’s plenty. It is very helpful when someone goes to fix an issue like this to have links, they don’t have to be precise - close it good enough. 😄

[sarora2073]

+1 on this.

re: why it’s blocked, perhaps it makes sense to have another brace-style option all together that is more blunt / simple in approach and just preserves the brace style period.