Document or change layout for MeanShareAndBalanceControls

[jessegreenberg]

Can you describe in the code how the layout works for the MeanShareAndBalanceControls? This value may be difficult to find, if the layout for the sim needs to be adjusted. Can you describe how 230 and 350 were found (is it just by inspection to align with the bottom of the notepad)? Alternatively, is there another way to set the layout for these components so that they are positioned to align relatively to other artwork in the ScreenVIew?

mean-share-and-balance/js/common/view/MeanShareAndBalanceControls.ts

Lines 112 to 116 in 1b60d3c

	this.numberSpinner = controlsAlignGroup.createBox( numberSpinner, {
	top: 350,
	xAlign: 'left'
	} );

mean-share-and-balance/js/common/view/MeanShareAndBalanceControls.ts

Lines 79 to 85 in 1b60d3c

	this.infoButton = new InfoButton( {
	listener: () => {
	options.infoPanelVisibleProperty!.value = !options.infoPanelVisibleProperty!.value;
	options.onInfoButtonPressed();
	},
	centerY: 230,
	scale: 0.6,

For #270.

[marlitas]

The top value for the numberSpinner is purely empirical based on the proportions of the controls and what seems to look good across all the screens. There's not one single thing that lines them up necessarily, but they must be in the same spot in all screens. I tried to add documentation to cover that, but let me know if it's not enough.

I went ahead and added the logic to calculate the vertical position of the info button instead of having it be empirical. Ready for your review @jessegreenberg!

[jessegreenberg]

I went ahead and added the logic to calculate the vertical position of the info button instead of having it be empirical

Nice, I think that is a great improvement. Thanks!

I edited the comment for the number spinner to indicate it is adjacent to the "real world" representation, I hope that is OK. Ready to close.