Move from the deprecated keyCode to key for keyboard events

Author: julienw

src/AbstractMenu.js

@@ -25,24 +25,29 @@ export default class AbstractMenu extends Component {
             return;
         }
 
-        switch (e.keyCode) {
-            case 37: // left arrow
-            case 27: // escape
+        switch (e.key) {
+            case 'ArrowLeft': // left arrow
+            case 'Left': // IE specific value
+            case 'Escape': // escape
+            case 'Esc': // IE specific value
                 e.preventDefault();
                 this.hideMenu(e);
                 break;
-            case 38: // up arrow
+            case 'ArrowUp': // up arrow
+            case 'Up': // IE specific value
                 e.preventDefault();
                 this.selectChildren(true);
                 break;
-            case 40: // down arrow
+            case 'ArrowDown': // down arrow
+            case 'Down': // IE specific value
                 e.preventDefault();
                 this.selectChildren(false);
                 break;
-            case 39: // right arrow
+            case 'ArrowRight': // right arrow
+            case 'Right': // IE specific value
                 this.tryToOpenSubMenu(e);
                 break;
-            case 13: // enter
+            case 'Enter': // enter
                 e.preventDefault();
                 this.tryToOpenSubMenu(e);
                 {

src/ContextMenu.js

@@ -170,7 +170,7 @@ export default class ContextMenu extends AbstractMenu {
     // Disabling this rule for more consistency.
     /* eslint-disable-next-line class-methods-use-this */
     hideMenu = (e) => {
-        if (e.keyCode === 27 || e.keyCode === 13) { // ECS or enter
+        if (e.key === 'Escape' || e.key === 'Esc' || e.key === 'Enter') {
             hideMenu();
         }
     };