phetsims / density

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

Reviewing Density PhET-iO example #158

Closed DianaTavares closed 1 year ago

DianaTavares commented 1 year ago

I already go throw all the Example document and everything is very well understandable!

I notice just one issue: In specify block mass - Mystery I think we already talked about this, user can build blocks of any size by changing its mass (no limited to the 1 - 10 L): image

If that is ok, and that is the reason of the text “Note that volume must be 1-10 L, so take care that the customized massProperty results in a volume within this range”, then you can close this issue.

Thanks!

arouinfar commented 1 year ago

Tagging #151.

Thanks @DianaTavares!

If that is ok, and that is the reason of the text “Note that volume must be 1-10 L, so take care that the customized massProperty results in a volume within this range”, then you can close this issue.

Unfortunately, this is a known issue reported in https://github.com/phetsims/density/issues/154. I included the warning so that clients are aware. PhET-iO has the power to create problematic situations, but we generally follow the principle of "customize wisely" and this is not a wise customization. :)