removed workaround code for fuzzBoard test issue, see #217

phetsims · Jan 20, 2021 · ceacf8b · ceacf8b
1 parent 558c30a
commit ceacf8b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/js/faradays-law/view/MagnetAutoSlideKeyboardListener.js b/js/faradays-law/view/MagnetAutoSlideKeyboardListener.js
@@ -165,11 +165,7 @@ class MagnetAutoSlideKeyboardListener {
       // check if the key is "modified"
       let keyModified = false;
       KEY_MODIFIER_LIST.forEach( modifierArg => {
-
-        // TODO: We need to check for whether getModifierState is defined, but so far it's only undefined during fuzzBoard testing, see https://github.com/phetsims/faradays-law/issues/217.
-        if ( event.domEvent.getModifierState ) {
-          keyModified = keyModified || event.domEvent.getModifierState( modifierArg );
-        }
+        keyModified = keyModified || event.domEvent.getModifierState( modifierArg );
       } );
 
       const keyCode = event.domEvent.keyCode;

diff --git a/js/faradays-law/view/MagnetNodeWithField.js b/js/faradays-law/view/MagnetNodeWithField.js
@@ -314,9 +314,7 @@ const createMagnetNode = magnet => {
  * @returns {boolean}
  */
 const isKeyModified = domEvent => {
-
-  // TODO: We need to check for whether getModifierState is defined, but so far it's only undefined during fuzzBoard testing, see https://github.com/phetsims/faradays-law/issues/217.
-  return domEvent.getModifierState && ( domEvent.getModifierState( 'Control' ) || domEvent.getModifierState( 'Alt' ) );
+  return domEvent.getModifierState( 'Control' ) || domEvent.getModifierState( 'Alt' );
 };
 
 faradaysLaw.register( 'MagnetNodeWithField', MagnetNodeWithField );