Put opening brace at the end of an existing line, see #88

phetsims · Mar 10, 2023 · 382d246 · 382d246
1 parent d3d7cf7
commit 382d246
Showing 1 changed file with 9 additions and 13 deletions.
diff --git a/js/common/view/MySolarSystemScreenView.ts b/js/common/view/MySolarSystemScreenView.ts
@@ -120,19 +120,15 @@ export default class MySolarSystemScreenView extends SolarSystemCommonScreenView
     } );
 
     const topRightControlBox = new AlignBox(
-      //REVIEW: Braces are on separate lines unnecessarily, and it forces another indent level. Put ( { on same line as VBox.
-      new VBox(
-        {
-          spacing: 10,
-          stretch: true,
-          children: [
-            new Panel( new Node( { children: [ labModeComboBox ], visible: model.isLab } ), SolarSystemCommonConstants.CONTROL_PANEL_OPTIONS ),
-            this.timeBox,
-            new Panel( checkboxesControlPanel, SolarSystemCommonConstants.CONTROL_PANEL_OPTIONS )
-          ]
-        }
-      ),
-      {
+      new VBox( {
+        spacing: 10,
+        stretch: true,
+        children: [
+          new Panel( new Node( { children: [ labModeComboBox ], visible: model.isLab } ), SolarSystemCommonConstants.CONTROL_PANEL_OPTIONS ),
+          this.timeBox,
+          new Panel( checkboxesControlPanel, SolarSystemCommonConstants.CONTROL_PANEL_OPTIONS )
+        ]
+      } ), {
         alignBoundsProperty: this.availableBoundsProperty,
         margin: SolarSystemCommonConstants.MARGIN,
         xAlign: 'right',