You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I took a look at this, but could not address it because the requirements are unclear. showMouseCueProperty lives in GroupSelectModel.ts, so changing it's instrumentation will affect other sims. It's unclear whether it should be uninstrumented for all sims, or GroupSelectModel needs an additional option to specify whether it's instrumented. And I'm guessing that this requires discussion outside the Number Pairs team.
I will also note that PhET typically avoids using "show" and "hide" in tandem names, and it's concerning to find it used here in common code. "Visible" is the preferred naming convention, in this case mouseCueVisibleProperty: Property<boolean>.
numberPairs.tenScreen.model.groupSelectBeadsModel.showMouseCueProperty
numberPairs.tenScreen.model.groupSelectLocationObjectsModel.showMouseCueProperty
The text was updated successfully, but these errors were encountered: