Bugfix nonprintable keys

src/const/NonPrintableKeysDictionary.js

@@ -0,0 +1,10 @@
+/**
+ * Dictionary of keys whose name is not a single symbol or character
+ */
+import dictionaryFrom from '../utils/object/dictionaryFrom';
+import translateToKey from '../vendor/react-dom/translateToKey';
+
+const NonPrintableKeysDictionary =
+  dictionaryFrom(Object.values(translateToKey), true);
+
+export default NonPrintableKeysDictionary;

src/helpers/parsing-key-maps/isCustomKeyName.js

@@ -0,0 +1,12 @@
+import Configuration from '../../lib/Configuration';
+
+/**
+ * Whether the specified key name is among those defined as custom key codes
+ * @param {ReactKeyName} keyName Name of the key
+ * @return {Boolean} true if keyName matches a custom key name
+ */
+function isCustomKeyName(keyName) {
+  return Configuration.option('_customKeyNamesDict')[keyName];
+}
+
+export default isCustomKeyName;

src/helpers/parsing-key-maps/isNonPrintableKeyName.js

@@ -0,0 +1,13 @@
+import NonPrintableKeysDictionary from '../../const/NonPrintableKeysDictionary';
+
+/**
+ * Whether the specified key is a valid key name that is not a single character or
+ * symbol
+ * @param {ReactKeyName} keyName Name of the key
+ * @returns {boolean} Whether the key is a valid special key
+ */
+function isNonPrintableKeyName(keyName) {
+  return !!NonPrintableKeysDictionary[keyName];
+}
+
+export default isNonPrintableKeyName;

src/helpers/parsing-key-maps/isValidKey.js

@@ -1,7 +1,10 @@
-import isSpecialKey from './isSpecialKey';
+import isNonPrintableKeyName from './isNonPrintableKeyName';
+import isCustomKeyName from './isCustomKeyName';
 
 function isValidKey(keyName) {
-  return isSpecialKey(keyName) || String.fromCharCode(keyName.charCodeAt(0)) === keyName;
+  return isNonPrintableKeyName(keyName) ||
+    String.fromCharCode(keyName.charCodeAt(0)) === keyName ||
+    isCustomKeyName(keyName)
 }
 
 export class InvalidKeyNameError extends Error {

src/helpers/resolving-handlers/hasKeyPressEvent.js

@@ -1,12 +1,12 @@
-import KeysWithoutPressEventDictionary from '../../const/KeysWithoutPressEventDictionary';
+import isNonPrintableKeyName from '../parsing-key-maps/isNonPrintableKeyName';
 
 /**
  * Whether the specified key name is for a key that has a native keypress event
  * @param {NormalizedKeyName} keyName Name of the key
  * @returns {Boolean} Whether the key has a native keypress event
  */
 function hasKeyPressEvent(keyName) {
-  return !KeysWithoutPressEventDictionary[keyName];
+  return !isNonPrintableKeyName(keyName);
 }
 
 export default hasKeyPressEvent;

src/lib/Configuration.js

@@ -144,12 +144,16 @@ class Configuration {
    * @see _configuration
    */
   static init(configuration) {
-    const {ignoreTags} = configuration;
+    const {ignoreTags, customKeyCodes } = configuration;
 
     if (ignoreTags) {
       configuration._ignoreTagsDict = dictionaryFrom(configuration.ignoreTags);
     }
 
+    if (customKeyCodes) {
+      configuration._customKeyNamesDict = dictionaryFrom(Object.values(configuration.customKeyCodes));
+    }
+
     if (process.env.NODE_ENV === 'production') {
       if (['verbose', 'debug', 'info'].indexOf(configuration.logLevel) !== -1) {
         console.warn(

src/lib/strategies/FocusOnlyKeyEventStrategy.js

@@ -361,7 +361,7 @@ class FocusOnlyKeyEventStrategy extends AbstractKeyEventStrategy {
       this._callHandlerIfActionNotHandled(event, _key, KeyEventRecordIndex.keydown, componentId, focusTreeId);
     }
 
-    this._simulateKeyPressesMissingFromBrowser(event, _key, focusTreeId, componentId, options);
+    this._simulateKeypressForNonPrintableKeys(event, _key, focusTreeId, componentId, options);
 
     this._updateEventPropagationHistory(componentId, componentPosition);
 
@@ -619,7 +619,7 @@ class FocusOnlyKeyEventStrategy extends AbstractKeyEventStrategy {
     }
   }
 
-  _simulateKeyPressesMissingFromBrowser(event, key, focusTreeId, componentId, options){
+  _simulateKeypressForNonPrintableKeys(event, key, focusTreeId, componentId, options){
     this._handleEventSimulation(
       'keypressEventsToSimulate',
       'simulatePendingKeyPressEvents',

src/lib/strategies/GlobalKeyEventStrategy.js

@@ -321,7 +321,7 @@ class GlobalKeyEventStrategy extends AbstractKeyEventStrategy {
       this._callHandlerIfExists(event, _key, KeyEventRecordIndex.keydown);
     }
 
-    this._simulateKeyPressesMissingFromBrowser(event, _key);
+    this._simulateKeypressForNonPrintableKeys(event, _key);
   }
 
   _howReactAppRespondedTo(event, key, eventRecordIndex) {
@@ -534,7 +534,7 @@ class GlobalKeyEventStrategy extends AbstractKeyEventStrategy {
     }
   }
 
-  _simulateKeyPressesMissingFromBrowser(event, key) {
+  _simulateKeypressForNonPrintableKeys(event, key) {
     this.keyEventManager.simulatePendingKeyPressEvents();
 
     this._handleEventSimulation(

src/vendor/react-dom/reactsGetEventKey.js

@@ -8,6 +8,7 @@
  */
 
 import getEventCharCode from './getEventCharCode';
+import translateToKey from './translateToKey';
 
 /**
  * Normalization of deprecated HTML5 `key` values
@@ -28,50 +29,6 @@ const normalizeKey = {
   MozPrintableKey: 'Unidentified',
 };
 
-/**
- * Translation from legacy `keyCode` to HTML5 `key`
- * Only special keys supported, all others depend on keyboard layout or browser
- * @see https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent#Key_names
- */
-const translateToKey = {
-  '8': 'Backspace',
-  '9': 'Tab',
-  '12': 'Clear',
-  '13': 'Enter',
-  '16': 'Shift',
-  '17': 'Control',
-  '18': 'Alt',
-  '19': 'Pause',
-  '20': 'CapsLock',
-  '27': 'Escape',
-  '32': ' ',
-  '33': 'PageUp',
-  '34': 'PageDown',
-  '35': 'End',
-  '36': 'Home',
-  '37': 'ArrowLeft',
-  '38': 'ArrowUp',
-  '39': 'ArrowRight',
-  '40': 'ArrowDown',
-  '45': 'Insert',
-  '46': 'Delete',
-  '112': 'F1',
-  '113': 'F2',
-  '114': 'F3',
-  '115': 'F4',
-  '116': 'F5',
-  '117': 'F6',
-  '118': 'F7',
-  '119': 'F8',
-  '120': 'F9',
-  '121': 'F10',
-  '122': 'F11',
-  '123': 'F12',
-  '144': 'NumLock',
-  '145': 'ScrollLock',
-  '224': 'Meta',
-};
-
 /**
  * @param {object} nativeEvent Native browser event.
  * @return {string} Normalized `key` property.
@@ -97,6 +54,7 @@ function reactsGetEventKey(nativeEvent) {
     // thus be captured by `keypress`, no other non-printable key should.
     return charCode === 13 ? 'Enter' : String.fromCharCode(charCode);
   }
+
   if (nativeEvent.type === 'keydown' || nativeEvent.type === 'keyup') {
     // While user keyboard layout determines the actual meaning of each
     // `keyCode` value, almost all function keys have a universal value.

src/vendor/react-dom/translateToKey.js

@@ -0,0 +1,45 @@
+/**
+ * Translation from legacy `keyCode` to HTML5 `key`
+ * Only special keys supported, all others depend on keyboard layout or browser
+ * @see https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent#Key_names
+ */
+const translateToKey = {
+  '8': 'Backspace',
+  '9': 'Tab',
+  '12': 'Clear',
+  '13': 'Enter',
+  '16': 'Shift',
+  '17': 'Control',
+  '18': 'Alt',
+  '19': 'Pause',
+  '20': 'CapsLock',
+  '27': 'Escape',
+  '32': ' ',
+  '33': 'PageUp',
+  '34': 'PageDown',
+  '35': 'End',
+  '36': 'Home',
+  '37': 'ArrowLeft',
+  '38': 'ArrowUp',
+  '39': 'ArrowRight',
+  '40': 'ArrowDown',
+  '45': 'Insert',
+  '46': 'Delete',
+  '112': 'F1',
+  '113': 'F2',
+  '114': 'F3',
+  '115': 'F4',
+  '116': 'F5',
+  '117': 'F6',
+  '118': 'F7',
+  '119': 'F8',
+  '120': 'F9',
+  '121': 'F10',
+  '122': 'F11',
+  '123': 'F12',
+  '144': 'NumLock',
+  '145': 'ScrollLock',
+  '224': 'Meta',
+};
+
+export default translateToKey;