Refactor keycode exporting in ui_framework

src/core_plugins/kibana/public/dashboard/dashboard.js

@@ -25,7 +25,7 @@ import { documentationLinks } from 'ui/documentation_links/documentation_links';
 import { showCloneModal } from './top_nav/show_clone_modal';
 import { migrateLegacyQuery } from 'ui/utils/migrateLegacyQuery';
 import { QueryManagerProvider } from 'ui/query_manager';
-import { ESC_KEY_CODE } from 'ui_framework/services';
+import { keycodes } from 'ui_framework/services';
 import { DashboardContainerAPI } from './dashboard_container_api';
 
 const app = uiModules.get('app/dashboard', [
@@ -328,7 +328,7 @@ app.directive('dashboardApp', function ($injector) {
       $scope.exitFullScreenMode = () => setFullScreenMode(false);
 
       document.addEventListener('keydown', (e) => {
-        if (e.keyCode === ESC_KEY_CODE) {
+        if (e.keyCode === keycodes.ESCAPE) {
           setFullScreenMode(false);
         }
       }, false);

src/core_plugins/kibana/public/management/sections/settings/advanced_row.js

@@ -33,7 +33,7 @@ uiModules.get('apps/management')
       };
 
       $scope.maybeCancel = function ($event, conf) {
-        if ($event.keyCode === keyCodes.ESC) {
+        if ($event.keyCode === keyCodes.ESCAPE) {
           $scope.cancelEdit(conf);
         }
       };

src/ui/public/accessibility/__tests__/kbn_ui_ace_keyboard_mode.js

@@ -5,7 +5,7 @@ import ngMock from 'ng_mock';
 import '../kbn_ui_ace_keyboard_mode';
 import {
   ENTER_KEY,
-  ESC_KEY_CODE,
+  keycodes,
 } from 'ui_framework/services';
 
 describe('kbnUiAceKeyboardMode directive', () => {
@@ -41,7 +41,7 @@ describe('kbnUiAceKeyboardMode directive', () => {
       const hint = element.find('.uiAceKeyboardHint');
       sinon.spy(hint[0], 'focus');
       const ev = angular.element.Event('keydown'); // eslint-disable-line new-cap
-      ev.keyCode = ESC_KEY_CODE;
+      ev.keyCode = keycodes.ESCAPE;
       textarea.trigger(ev);
       expect(hint[0].focus.called).to.be(true);
       expect(hint.hasClass('uiAceKeyboardHint-isInactive')).to.be(false);
@@ -91,7 +91,7 @@ describe('kbnUiAceKeyboardModeService', () => {
       const hint = element.find('.uiAceKeyboardHint');
       sinon.spy(hint[0], 'focus');
       const ev = angular.element.Event('keydown'); // eslint-disable-line new-cap
-      ev.keyCode = ESC_KEY_CODE;
+      ev.keyCode = keycodes.ESCAPE;
       textarea.trigger(ev);
       expect(hint[0].focus.called).to.be(true);
       expect(hint.hasClass('uiAceKeyboardHint-isInactive')).to.be(false);

src/ui/public/accessibility/kbn_ui_ace_keyboard_mode.js

@@ -15,7 +15,7 @@
 import angular from 'angular';
 import { uiModules } from 'ui/modules';
 import './kbn_ui_ace_keyboard_mode.less';
-import { ESC_KEY_CODE, ENTER_KEY } from 'ui_framework/services';
+import { keycodes, ENTER_KEY } from 'ui_framework/services';
 
 let aceKeyboardModeId = 0;
 
@@ -64,7 +64,7 @@ uiModules.get('kibana')
     });
 
     uiAceTextbox.keydown((ev) => {
-      if (ev.keyCode === ESC_KEY_CODE) {
+      if (ev.keyCode === keycodes.ESCAPE) {
         ev.preventDefault();
         ev.stopPropagation();
         enableOverlay();

ui_framework/src/components/modal/confirm_modal.js

@@ -8,7 +8,7 @@ import { KuiModalHeaderTitle } from './modal_header_title';
 import { KuiModalBody } from './modal_body';
 import { KuiModalBodyText } from './modal_body_text';
 import { KuiButton } from '../index';
-import { ESC_KEY_CODE } from '../../services';
+import { keycodes } from '../../services';
 
 export const CONFIRM_BUTTON = 'confirm';
 export const CANCEL_BUTTON = 'cancel';
@@ -32,7 +32,7 @@ export function KuiConfirmModal({
 
   const onKeyDown = (event) => {
     // Treat the 'esc' key as a cancel indicator.
-    if (event.keyCode === ESC_KEY_CODE) {
+    if (event.keyCode === keycodes.ESCAPE) {
       onCancel();
     }
   };

ui_framework/src/services/accessibility/combo_box_key_codes.js

@@ -8,17 +8,17 @@
  */
 
 import {
-  DOWN_KEY_CODE,
-  ENTER_KEY_CODE,
-  ESC_KEY_CODE,
-  TAB_KEY_CODE,
-  UP_KEY_CODE,
+  DOWN,
+  ENTER,
+  ESCAPE as ESC,
+  TAB,
+  UP,
 } from '../key_codes';
 
 export const comboBoxKeyCodes = {
-  DOWN: DOWN_KEY_CODE,
-  ENTER: ENTER_KEY_CODE,
-  ESC: ESC_KEY_CODE,
-  TAB: TAB_KEY_CODE,
-  UP: UP_KEY_CODE,
+  DOWN,
+  ENTER,
+  ESC,
+  TAB,
+  UP,
 };

ui_framework/src/services/index.js

@@ -1,3 +1,7 @@
+// Export all keycodes under a `keycodes` named variable
+import * as keycodes from './key_codes';
+export { keycodes };
+
 export {
   accessibleClickKeys,
   comboBoxKeyCodes,
@@ -6,6 +10,5 @@ export {
 } from './accessibility';
 
 export { SortableProperties } from './sort';
-export { ESC_KEY_CODE } from './key_codes';
 
 export { LEFT_ALIGNMENT, RIGHT_ALIGNMENT } from './alignment';

ui_framework/src/services/key_codes.js

@@ -1,5 +1,7 @@
-export const DOWN_KEY_CODE = 40;
-export const ENTER_KEY_CODE = 13;
-export const ESC_KEY_CODE = 27;
-export const TAB_KEY_CODE = 9;
-export const UP_KEY_CODE = 38;
+export const ENTER = 13;
+export const ESCAPE = 27;
+export const TAB = 9;
+
+// Arrow keys
+export const DOWN = 40;
+export const UP = 38;