add isModal explicitly to looker's state.config

use it to reliably stopPropagation of the Escape key event

Author: jleven

app/packages/looker/src/elements/common/looker.ts

@@ -30,6 +30,7 @@ export class LookerElement<State extends BaseState> extends BaseElement<
             SHORTCUTS,
             error,
             options: { shouldHandleKeyEvents },
+            config: { isModal },
           } = state;
           if (!error && e.key in SHORTCUTS) {
             const matchedControl = SHORTCUTS[e.key] as Control;
@@ -38,9 +39,6 @@ export class LookerElement<State extends BaseState> extends BaseElement<
             if (enabled) {
               matchedControl.action(update, dispatchEvent, e.key, e.shiftKey);
 
-              // If this is a Modal, we don't want Escape to bubble up to the Grid
-              // Hack: `lookerOptions` only sets showControl=true when a Modal
-              const isModal = state.options.showControls === true;
               if (e.key === "Escape" && isModal) {
                 e.stopPropagation();
               }

app/packages/looker/src/state.ts

@@ -222,6 +222,7 @@ export interface BaseConfig {
     id: string;
     name: string;
   };
+  isModal?: boolean;
 }
 
 export interface FrameConfig extends BaseConfig {

app/packages/state/src/hooks/useCreateLooker.ts

@@ -159,6 +159,7 @@ export default <T extends AbstractLooker<BaseState>>(
           thumbnail,
           view,
           shouldHandleKeyEvents: isModal,
+          isModal,
         };
 
         let sampleMediaFilePath = urls[mediaField];