From 551487ec59c9ce0a282f1d211c0554eb91bf7fe1 Mon Sep 17 00:00:00 2001 From: Marla Schulz Date: Wed, 5 Feb 2025 15:44:44 -0800 Subject: [PATCH] Add condition to update proposed bead position only when group item is keyboard grabbed, see: https://github.com/phetsims/number-pairs/issues/62 --- js/common/view/BeadsOnWireNode.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/js/common/view/BeadsOnWireNode.ts b/js/common/view/BeadsOnWireNode.ts index 9e16af1..ea50dfd 100644 --- a/js/common/view/BeadsOnWireNode.ts +++ b/js/common/view/BeadsOnWireNode.ts @@ -376,8 +376,10 @@ export default class BeadsOnWireNode extends Node { } ); // Update the proposed bead position if it does not match the grabbed bead's position. - this.keyboardProposedBeadPositionProperty.value.x !== grabbedBeadNode.countingObject.beadXPositionProperty.value && - this.keyboardProposedBeadPositionProperty.set( new Vector2( grabbedBeadNode.countingObject.beadXPositionProperty.value, 0 ) ); + if ( this.model.groupSelectBeadsModel.isGroupItemKeyboardGrabbedProperty.value ) { + this.keyboardProposedBeadPositionProperty.value.x !== grabbedBeadNode.countingObject.beadXPositionProperty.value && + this.keyboardProposedBeadPositionProperty.set( new Vector2( grabbedBeadNode.countingObject.beadXPositionProperty.value, 0 ) ); + } assert && assert( this.leftAddendCountingObjectsProperty.value.length === this.model.leftAddendProperty.value, 'leftAddendObjects.length should match leftAddendNumberProperty' ); assert && assert( this.rightAddendCountingObjectsProperty.value.length === this.model.rightAddendProperty.value, 'rightAddendObjects.length should match rightAddendNumberProperty' );