-
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
GrabDragInteraction should factor out model for reuse #867
Labels
Comments
I got to a commit point:
|
zepumph
added a commit
that referenced
this issue
Aug 27, 2024
Signed-off-by: Michael Kauzmann <[email protected]>
zepumph
added a commit
to phetsims/density-buoyancy-common
that referenced
this issue
Aug 27, 2024
…phet#867 Signed-off-by: Michael Kauzmann <[email protected]>
zepumph
added a commit
to phetsims/density-buoyancy-common
that referenced
this issue
Aug 28, 2024
…onState too, phetsims/scenery-phet#867 Signed-off-by: Michael Kauzmann <[email protected]>
zepumph
added a commit
that referenced
this issue
Aug 28, 2024
…onState too, #867 Signed-off-by: Michael Kauzmann <[email protected]>
Noting that some of the above may overlap with #869. Note that issue too. |
All items have been checked above. Not all because they are done; many will be handled over in #869. Closing |
zepumph
added a commit
that referenced
this issue
Sep 3, 2024
Signed-off-by: Michael Kauzmann <[email protected]>
zepumph
added a commit
to phetsims/density-buoyancy-common
that referenced
this issue
Sep 3, 2024
Signed-off-by: Michael Kauzmann <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From phetsims/density-buoyancy-common#368, we keep a Mass model instance around for multiple copies of the MassView (which creates the GrabDragInteraction). So let's make a grabDragInteractionModel that can be persistent through different grabDragInteraction instances. And perhaps such that multiple views can update the same model (for cue logic). We will see!
The text was updated successfully, but these errors were encountered: