lengthProperty should also respect hasListenerOrderDependencies, phet…

…sims/build-a-nucleus#105
phetsims · Aug 25, 2023 · 4f571cb · 4f571cb
1 parent e87e8b6
commit 4f571cb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/js/TinyEmitter.ts b/js/TinyEmitter.ts
@@ -70,6 +70,7 @@ export default class TinyEmitter<T extends TEmitterParameter[] = []> implements
       this.hasListenerOrderDependencies = hasListenerOrderDependencies;
     }
 
+    // Listener order is preserved in Set
     this.listeners = new Set();
 
     this.emitContexts = [];

diff --git a/js/createObservableArray.ts b/js/createObservableArray.ts
@@ -134,7 +134,8 @@ const createObservableArray = <T>( providedOptions?: ObservableArrayOptions<T> )
   const lengthProperty = new NumberProperty( 0, combineOptions<NumberPropertyOptions>( {
     numberType: 'Integer',
     tandem: options.tandem?.createTandem( 'lengthProperty' ),
-    phetioReadOnly: true
+    phetioReadOnly: true,
+    hasListenerOrderDependencies: options.hasListenerOrderDependencies
   }, options.lengthPropertyOptions ) );
 
   // The underlying array which is wrapped by the Proxy