stick with using Emitter when list box fires, #405

phetsims · Jan 22, 2019 · 4eacb80 · 4eacb80
1 parent 499f06b
commit 4eacb80
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/js/ComboBoxListBox.js b/js/ComboBoxListBox.js
@@ -52,9 +52,10 @@ define( require => {
 
       }, options );
 
-      // TODO sun#405 It seems it would be better to use FireListener on each ComboBoxListItemNode
       const firedEmitter = new Emitter( {
         argumentTypes: [ { validValue: Event } ],
+
+        // Pops down the list box and sets the property.value to match the chosen item.
         listener: event => {
 
           const listItemNode = event.currentTarget;