Additional key checks for browser compatibility (#2539)

Not all browsers recognize e.key === 'Escape'; some use 'Esc' and some only respond to the keyCode.
2017-04-30 09:12:14 -04:00 · 2017-04-30 09:12:14 -04:00 · 4d4d71cd67
parent ad3a6badb4
commit 4d4d71cd67
1 changed files with 2 additions and 1 deletions
--- a/app/assets/javascripts/components/features/ui/components/modal_root.jsx
+++ b/app/assets/javascripts/components/features/ui/components/modal_root.jsx
@ -22,7 +22,8 @@ class ModalRoot extends React.PureComponent {
  }

  handleKeyUp (e) {
-    if (e.key === 'Escape' && !!this.props.type) {
+    if ((e.key === 'Escape' || e.key === 'Esc' || e.keyCode === 27)
+         && !!this.props.type) {
      this.props.onClose();
    }
  }