-
Notifications
You must be signed in to change notification settings - Fork 6
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
Diffusion Screen: make it possible to hide one type of particles #255
Comments
For discussion with @arouinfar and @Nancy-Salpepi on 6/11/24, I added Setting its value to Things that I don't like about this: |
Reviewed with @arouinfar and @Nancy-Salpepi, looks good as is. @arouinfar has a note about adding this to examples.md, so we can close. |
For #30
It's not possible to limit students to a single type of particle on the Diffusion Screen. @Nancy-Salpepi noted that she usually has students explore how a single type of particles spreads out throughout the container.
In the
settingsPanel
, we can hide individual rows of controls for variables outside the scope of the lesson's goals.However, this prevents exploration of a single type of particles dispersing throughout the container, as we decided to uninsturment
numberSpinner.visibleProperty
.An easy solution would be to re-instrument the NumberSpinners for type 2 particles. The UX would be a bit clunky though, as it would require users to hide 4 controls. It's also an imperfect solution because the Data readouts and Center of Mass icon still refer to the hidden particle type.
@pixelzoom could we create a Property that would hide everything associated with type 2 particles? Let's discuss when you return.
The text was updated successfully, but these errors were encountered: