-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Min and Max mass listed don't take material into consideration #407
Comments
Actually, this looks like it's already addressed in the examples doc, so it can probably be closed. |
I asked about this same thing in the Buoyancy slack channel! Sam Reid Kathy Perkins Amy Rouinfar Yes, that behavior is fine. The examples clarifies that resultant volume needs to be 1-10L and recommends clients first refer to enabledRangeProperty before setting mass. Nancy Salpepi Wondering if that statement in the examples should come before the massProperty phetio IDs or do you think it is OK as is |
Ahh, thanks |
Ok great. Yes I believe all the actionables are in #405 and examples.md. Closing. |
Oh, we need to update examples. @arouinfar can you add this sentiment from the above comment:
|
Device
Samsung
OS
Win 11
Browser
Chrome
Problem Description
For phetsims/qa#1141
In studio, the mass ranges given for blocks may not take into account the material the block is currently made of. For example, the range for
buoyancy.exploreScreen.view.blockAPanel.massNumberControl.massProperty
is .1-27, but depending on the material is usually smaller than that.Visuals
The text was updated successfully, but these errors were encountered: