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 · c67d3c990b
parent 2e47fe3e1a
commit c67d3c990b
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();
    }
  }