Add extra loading hook, and remove close call for typonjs compat Fixes #89

Author: Posnet

README.md

@@ -82,6 +82,10 @@ Hooks.callAll("PopOut:popout", app, popout);
 // node: is the html element of the application after it has been moved to the new window.
 Hooks.callAll("Popout:loaded", app, node);
 
+// app: is the foundry application being popped out.
+// popout: is the browser window object where the popped out element will be moved.
+Hooks.callAll("Popout:loading", app, popout);
+
 // app: is the foundry application being popped in.
 Hooks.callAll("PopOut:popin", app);
 

popout.js

@@ -577,6 +577,8 @@ class PopoutModule {
 
     // -------------------- Add unload handlers --------------------
 
+    Hooks.callAll("PopOut:loading", app, popout); // eslint-disable-line no-undef
+
     window.addEventListener("unload", async (event) => {
       this.log("Unload event", event);
       const appId = app.appId;
@@ -837,7 +839,6 @@ class PopoutModule {
       } else {
         if (game.keyboard.downKeys.has("Escape")) return; // eslint-disable-line no-undef
       }
-      popout.close();
       return oldClose.apply(app, args);
     };