From 382d246a3c89fd66a04f7c13a313f5e6b815b0de Mon Sep 17 00:00:00 2001 From: Sam Reid Date: Thu, 9 Mar 2023 21:53:15 -0700 Subject: [PATCH] Put opening brace at the end of an existing line, see https://github.com/phetsims/my-solar-system/issues/88 --- js/common/view/MySolarSystemScreenView.ts | 22 +++++++++------------- 1 file changed, 9 insertions(+), 13 deletions(-) diff --git a/js/common/view/MySolarSystemScreenView.ts b/js/common/view/MySolarSystemScreenView.ts index b2ddd9d1..5d67e135 100644 --- 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',