search

phetsims / isotopes-and-atomic-mass

"Isotopes And Atomic Mass" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
3 stars 3 forks source link

Consider converting to default layoutBounds before PhET-iO publication #110

Closed pixelzoom closed 11 months ago

pixelzoom commented 1 year ago

This sim currently uses non-default layoutBounds.

In MakeIsotopesScreenView:

    super( { layoutBounds: ShredConstants.LAYOUT_BOUNDS } );

And in ShredConstants:

  LAYOUT_BOUNDS: new Bounds2( 0, 0, 768, 464 ),

What's especially concerning here is that shred is a common-code dependency, and it's using non-default layoutBounds. See https://github.com/phetsims/shred/issues/37

Per https://github.com/phetsims/phet-io/issues/1939... layoutBounds should not be changed for published PhET-iO sims because doing so may disrupt client customizations. So evaluate whether to change layoutBounds before publishing a PhET-iO version.

If you decide not to change layoutBounds, document why in MakeIsotopesScreenView, and refer to this issue.

marlitas commented 11 months ago

Closing per dev meeting request 12/7/23.