*** Bug 218843 has been marked as a duplicate of this bug. ***

*** Bug 221866 has been marked as a duplicate of this bug. ***

Thanks,petr.
Is there any plan to fix it?

(In reply to ifishing from comment #3)
> Thanks,petr.
> Is there any plan to fix it?
Not for the 7.4.

Hi, I was wondering if there is any plan to work on this in 8?

Netbeans 8 is already out, but I second Jock's motion: can we please fix this issue already? The misplaced braces are jarring.

Specifically, (as mention in issue #221866) it's not clear to me why you're indenting braces for switch-case statements differently for Javascript than Java. This "default" seems to contradict every single convention I've ever seen, so whether braces are configurable or not this should get fixed.

This is still an issue with the JavaScript formatting in 8.02. I've gotta say that the formatting options for JS is extremely lacking in comparison to Eclipse, I have to pull the file into Eclipse just to format it using my company's very specific format. Can we add more formatting support?

I agree with Jock,

This bug is extremely problematic if you have to adhere to company formatting standards.

Jock, I set Version back to 7.4 because it is meant to denote when the issue was first sighted (not the latest version it is reproducible in).

Please vote for this issue if you want it fixed sooner.

Indeed, for JavaScript, the formatting options are really poor compared to other languages.

Please add the same level of formatting options as for other languages. 

Thanks a lot, 

Best regards, 

Alexandre 8)

Adding options and fine tuning formatting is mainly about the time. I have spend probably 6 month with the implementing all the options in PHP formatter. I'm also for the implementation of the bulk of options in JS editor. But I have to admit that there is not so much time now. So we have to be patient.  On the other hand I think that the editor is the most part of the IDE, so I'm also voting for this issue.

Changeset: 7675a379a103
Author:    Roman Svitanic <rsvitanic@netbeans.org>
Date:      2015-01-22 16:44
Message:   #229334 - JavaScript Braces Placement formatting

Don't understand why this issue is being marked as RESOLVED/FIXED, the issue is still present in NetBeans 8.0.2

In Options/Editor/Formatting when choosing Language: Javascript, there is still no "Braces" option in the Category selector.

For javascript the only categories available are: Tabs and Indents, Spaces and Wrapping

You can see the meaning of "Status" at https://netbeans.org/bugzilla/page.cgi?id=fields.html#bug_status 
Note that the Target Milestone for this issue is Development.

The issue was marked as FIXED because the engineer added/modified code to "Fix" this issue. Normally, such changes will propagate through the NetBeans build system, and in a day (typically) appear in the Development distributions of NetBeans at http://bits.netbeans.org/download/trunk/nightly/latest/

At least, that's the way it usually works. So, if you want to see the result of the change, you'll probably need to download a NetBeans Development distribution in a day or two.

Roman,

Thank you very much! I'm going to try this out as soon as the next dev build comes out.

Integrated into 'main-silver', will be available in build *201501230002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/7675a379a103
User: Roman Svitanic <rsvitanic@netbeans.org>
Log: #229334 - JavaScript Braces Placement formatting
Task #229334 - Please add support for javascript braces formatting

Roman and Lou,

I have one outstanding question about this fix.

Bug #221866 (which is marked duplicate of this issue) complains that code-format is transforming:

switch (error.code)
{
	case error.PERMISSION_DENIED:
	{
			break;
	}
}

to:

switch (error.code)
{
	case error.PERMISSION_DENIED:
		{
			break;
		}
}

With the new Javascript formatting options, I can fix the above by changing "Continuation Indentation" from 2 to 0.

Are you sure that this behavior is appropriate? Does it really make sense for a case block to count as a "continuation"? I thought that a continuation is defined as a single line of code that spans multiple physical lines.

The Javascript formatting options do not match the behavior of the Java formatting options. In the latter, a continuation of 2 does not result in a case block getting indented.

Please let me know what you think.

In fact, I am pretty sure that this fix did not change the behavior for case statements at all.

Is it possible to reopen bug #221866?

_ gtzabari: Could you please attach here full code, where it does not work and perhaps exported IDE settings? I tried it and it works for me just fine, the formatted result is

try {

} catch (error) {
    switch (error.code)
    {
        case error.PERMISSION_DENIED:
        {
            break;
        }
    }
}


Thank you


Product Version: NetBeans IDE Dev (Build 201501260002)
Java: 1.8.0_25; Java HotSpot(TM) Client VM 25.25-b02
Runtime: Java(TM) SE Runtime Environment 1.8.0_25-b17
System: Linux version 3.13.0-35-generic running on i386; UTF-8; en_US (nb)

Vladimir,

My mistake. The problems occurs with Object.defineProperty(), not switch blocks. Given:

Object.defineProperty(this, "uri",
{
	value: uri,
	enumerable: true
});

Code-format converts it to:

Object.defineProperty(this, "uri",
	{
		value: uri,
		enumerable: true
	});

I am expecting it to retain the original formatting. The equivalent Java code (lambda) looks like this:

new Thread(() ->
{
	System.out.println("I am a runnable");
});

If you apply code-format on it, the original indentation will be kept (the block is not treated as a continuation.

Let me know how you want to proceed.

Vladimir,

Upon further investigation, I don't think any action is required for this issue. The bug I discussed above is already covered by bug #227007 which is still open.

Thanks a lot for providing these additional features, it is a great improvement.

Nevertheless, are there additional plans to add the following formatting capabilities:

* Alignment: new lines before statements else, elseif, catch, etc.
* Blank lines: introduction blank lines before / after functions, etc.

Thanks a lot, 

Best regards,

Alexandre 8)

Still present in Build 201411181905 (8.0.2). No options for configuration of braces for javascript whatsoever.

(In reply to Tomasko from comment #22)
> Still present in Build 201411181905 (8.0.2). No options for configuration of
> braces for javascript whatsoever.

Please do not reopen issues until you are sure that issue is not fixed. Look at the field "Target Milestone" which indicates the first version containing the fix. In this case it's "Dev" which means development/nightly builds. The next release containing this feature will be NetBeans 8.1.

This is indeed very frustrating and unfortunate since 2 years have passed without fixing the issue. At least I hope that you will bother to fix it in the upcoming NB release.

mkotsollaris,

Did you read the comment above yours before posting this? What version are you saying this is not fixed in?

mkotsollaris: This feature will be part of upcoming NetBeans 8.1. You can try daily Dev build from http://bits.netbeans.org/download/trunk/nightly/latest/.

Thank you