added lib/preview-frame-interface.ts. Fixes #44.

Author: toolness

lib/preview-frame-interface.ts

@@ -0,0 +1,20 @@
+// This needs to be kept in its own file because we want to be able to
+// import it from different source bundles without having to actually
+// bring in any implementation.
+
+// An alternative would be to use a .d.ts file for this, but we'd like
+// to be explicit about importing these interfaces so that the source
+// code is easier to understand.
+
+export interface PreviewFrameErrorReporter {
+  (message: string, line?: number): any
+}
+
+// Eventually we might want the preview frame to exist on a separate
+// origin for security, which means that we'd have to use postMessage()
+// to communicate with it. Thus this interface needs to be asynchronous.
+export interface PreviewFrame extends Window {
+  startSketch: (sketch: string, p5version: string, maxRunTime: number,
+                loopCheckFuncName: string, baseURL: string,
+                errorCb: PreviewFrameErrorReporter) => void
+}

lib/preview-frame.ts

@@ -1,3 +1,6 @@
+import { PreviewFrame,
+         PreviewFrameErrorReporter } from "./preview-frame-interface";
+
 require("../css/preview-frame.css");
 
 interface PreviewFrameWindow extends PreviewFrame {

lib/preview.tsx

@@ -4,6 +4,8 @@ import falafel from "./falafel";
 import LoopInserter from "./loop-inserter";
 import makeImplicitSketch from "./implicit-sketch";
 import PureComponent from "./pure-component";
+import { PreviewFrame,
+         PreviewFrameErrorReporter } from "./preview-frame-interface";
 
 const LOOP_CHECK_FUNC_NAME = '__loopCheck';
 

tsconfig.json

@@ -20,6 +20,7 @@
     "lib/autosaver.ts",
     "lib/p5-widget.ts",
     "lib/preview-frame.ts",
+    "lib/preview-frame-interface.ts",
     "lib/falafel.ts",
     "lib/implicit-sketch.ts",
     "lib/loop-inserter.ts",

typings/custom.d.ts

@@ -11,16 +11,3 @@ declare namespace Url {
 declare module "url" {
     export = Url;
 }
-
-declare interface PreviewFrameErrorReporter {
-  (message: string, line?: number): any
-}
-
-// Eventually we might want the preview frame to exist on a separate
-// origin for security, which means that we'd have to use postMessage()
-// to communicate with it. Thus this interface needs to be asynchronous.
-declare interface PreviewFrame extends Window {
-  startSketch: (sketch: string, p5version: string, maxRunTime: number,
-                loopCheckFuncName: string, baseURL: string,
-                errorCb: PreviewFrameErrorReporter) => void
-}