This case is not covered as an exception in the [style guide](http://contribute.jquery.org/style-guide/js).

I certainly agree that the closing brace of the fn looks better flush to
the right paren, but perhaps this invocation bears being called out
because technically it is:

`test( "some condition", fn(){}); `

were this any other pair of arguments:

`test( "foo", "bar" );`

or

`test( fn(){}, fn(){} );`

would be expected (wouldn't it?).

As you suggest here it is:

`test( "foo", fn(){});`

which doesn't fit into any of the rules or exceptions presently in the
guide.


On Wed, May 01, 2013 at 05:37:48AM -0700, Jörn Zaefferer wrote:
>> +
>> +	var element = $( "#bar1" ).menubar(),
>> +		debugDelay = 0,
>> +		firstMenuItem = $( "#bar1 .ui-menubar-item .ui-button:first" );
>> +
>> +	equal( element.find( ":tabbable").length, 1, "A Menubar should have 1 tabbable element on init" );
>> +	firstMenuItem[ 0 ].focus();
>> +
>> +	ok( firstMenuItem.hasClass( "ui-state-focus" ), "After a focus event, the first element should have the focus class" );
>> +	$( document.activeElement ).simulate( "keydown", { keyCode: $.ui.keyCode.LEFT } );
>> +
>> +	ok( !firstMenuItem.hasClass( "ui-state-focus" ), "After a keypress event, the first element should no longer have the focus class" );
>> +	ok( firstMenuItem.parent().siblings().last().children().eq( 0 ).hasClass( "ui-state-focus" ), "After a LEFT cursor event from  the first element, the last element should have focus" );
>> +
>> +	start();
>> +} );
>
>No space here, should be `});`
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/jquery/jquery-ui/pull/953/files#r4035702