Mass change workaround, see #328 and phetsims/scenery-phet#825

(cherry picked from commit 18cc9b3)
phetsims · Jan 10, 2024 · cf1e8a9 · cf1e8a9
1 parent f186f52
commit cf1e8a9
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/js/common/model/MySolarSystemModel.ts b/js/common/model/MySolarSystemModel.ts
@@ -149,6 +149,14 @@ export default class MySolarSystemModel extends SolarSystemCommonModel {
             this.saveStartingBodyInfo();
           }
         } );
+
+      body.massProperty.link( mass => {
+        // This workaround is because NumberControl endCallback fires before changing the property, at least on single presses
+        // TODO: Remove this workaround when https://github.com/phetsims/scenery-phet/issues/825 is fixed
+        if ( !body.userIsControllingMassProperty.value ) {
+          this.saveStartingBodyInfo();
+        }
+      } );
     } );
 
     this.timeProperty.lazyLink( time => {