phetsims / collision-lab

"Collision Lab" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
6 stars 4 forks source link

CollisionLabTimeControlNode.playPauseStepButtonOptions pass the same values as defaults #185

Closed zepumph closed 4 years ago

zepumph commented 4 years ago

In CollisionLabTimeControlNode I see:

      playPauseStepButtonOptions: {
        playPauseStepXSpacing: 9,
        playPauseButtonOptions: { radius: 30 },
        stepBackwardButtonOptions: { radius: 20 },
        stepForwardButtonOptions: { radius: 20 }
      },

The radius for stepButton by default is 20, and the radius for the play pause button is 28. Is that close enough? I don't see the value of maintaining these hard coded values in this sim, but I'm also an outsider. Feel free to do whatever is best!

I go here while working on https://github.com/phetsims/sun/issues/657

jonathanolson commented 4 years ago

Removed those, since the defaults seem close enough. Thank you!