Merge pull request w3c#1 from jeonghee27/polyfil-bugfix

Fixed a bug of Enter key behavior.
frivoal · Aug 20, 2018 · 7d8d39c · 7d8d39c
2 parents c6007b7 + c9e92f2
commit 7d8d39c
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/polyfill/spatnav-heuristic.js b/polyfill/spatnav-heuristic.js
@@ -46,7 +46,7 @@ function focusNavigationHeuristics(spatnavPolyfillOptions) {
   * mouseclick EventListener :
   * If the mouse click a point in the page, the point will be the starting point.
   */
-  document.addEventListener('click', function(e) {
+  document.addEventListener('mouseup', function(e) {
     startingPosition = {xPosition: e.clientX, yPosition: e.clientY};
   });
 
@@ -992,6 +992,10 @@ function focusNavigationHeuristics(spatnavPolyfillOptions) {
     const focusNavigableArrowKey = {'left': false, 'up': false, 'right': false, 'down': false};
 
     const dir = ARROW_KEY_CODE[e.keyCode];
+    if(dir === undefined) {
+      return focusNavigableArrowKey;
+    }
+
     if (spinnableInputTypes.includes(eventTarget.getAttribute('type')) &&
       (dir === 'up' || dir === 'down')) {
       focusNavigableArrowKey[dir] = true;