Escape no longer closes the modal (#5866)

Author: dimitardanailov

src/components/App.js

@@ -12,6 +12,15 @@ import actions from "../actions";
 import { ShortcutsModal } from "./ShortcutsModal";
 import VisibilityHandler from "./shared/VisibilityHandler";
 
+import SplitBox from "devtools-splitter";
+import ProjectSearch from "./ProjectSearch";
+import PrimaryPanes from "./PrimaryPanes";
+import Editor from "./Editor";
+import SecondaryPanes from "./SecondaryPanes";
+import WelcomeBox from "./WelcomeBox";
+import EditorTabs from "./Editor/Tabs";
+import QuickOpenModal from "./QuickOpenModal";
+
 import {
   getSelectedSource,
   getPaneCollapse,
@@ -139,17 +148,18 @@ class App extends Component<Props, State> {
   onEscape = (_, e) => {
     const {
       activeSearch,
-      quickOpenEnabled,
       closeActiveSearch,
-      closeQuickOpen
+      closeQuickOpen,
+      quickOpenEnabled
     } = this.props;
 
     if (activeSearch) {
       e.preventDefault();
       closeActiveSearch();
     }
 
-    if (quickOpenEnabled === true) {
+    if (quickOpenEnabled) {
+      e.preventDefault();
       closeQuickOpen();
     }
   };