add onInputValueChange to AccessiblePeer for phetsims/sun#397

phetsims · Sep 11, 2018 · 4a419ee · 4a419ee
1 parent 1325608
commit 4a419ee
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/js/accessibility/Accessibility.js b/js/accessibility/Accessibility.js
@@ -1892,7 +1892,10 @@ define( function( require ) {
           value = '' + value;
           this._inputValue = value;
 
-          this.setAccessibleAttribute( 'value', value );
+          for ( var i = 0; i < this.accessibleInstances.length; i++ ) {
+            var peer = this.accessibleInstances[ i ].peer;
+            peer.onInputValueChange();
+          }
         },
         set inputValue( value ) { this.setInputValue( value ); },
 

diff --git a/js/accessibility/AccessiblePeer.js b/js/accessibility/AccessiblePeer.js
@@ -367,6 +367,14 @@ define( function( require ) {
       }
     },
 
+    /**
+     * Set the input value on the peer's primary sibling element. Using the value setter seems to be the only way
+     * to have the value be set correctly so we cannot use onAttributeChange for this.
+     */
+    onInputValueChange: function() {
+      this.primarySibling.value = this.node.inputValue;
+    },
+
     /**
      * Called when our parallel DOM element gets focused.
      * @private