@raydiant/playlist-window-manager 0.28.1 → 0.28.2-rc2

package/build/BrowserWindowRenderer.js

@@ -128,6 +128,7 @@ var BrowserWindowRenderer = /** @class */ (function () {
                                     },
                                 })
                                 : this.createOrGetPooledBrowserWindow(windowId, {
+                                    paintWhenInitiallyHidden: false,
                                     show: false,
                                     /**
                                      * This option fixes window not being able to take fullscreen leaving

package/build/PlaylistWindowManager.d.ts

@@ -17,6 +17,7 @@ export interface PlaylistWindowManagerOptions {
     parentWindowId?: string;
     windowName?: string;
     isPreview?: boolean;
+    closeWindowDelayMs?: number;
     onEnableInterruptLock?: () => void;
     onDisableInterruptLock?: () => void;
     getTerminal?: () => Terminal | null;

package/build/PlaylistWindowManager.js

@@ -74,6 +74,7 @@ var WindowLoadError_1 = __importDefault(require("./WindowLoadError"));
 // Wait at least 3 seconds before restarting or moving onto the next window
 // to prevent log spam causing devices to become unresponsive.
 var minTimeBetweenCompletes = 3000;
+var delay = function (ms) { return new Promise(function (res) { return setTimeout(res, ms); }); };
 var PlaylistWindowManager = /** @class */ (function (_super) {
     __extends(PlaylistWindowManager, _super);
     function PlaylistWindowManager(options) {
@@ -123,7 +124,7 @@ var PlaylistWindowManager = /** @class */ (function (_super) {
             _this.emitCurrentWindowEvent('window_charge_request', request);
         };
         _this.handleCurrentPresentationComplete = function () { return __awaiter(_this, void 0, void 0, function () {
-            var logger, isThrottled, now, timeSinceLastComplete, delay, _a, next, preview, nextPreviewPresentationId;
+            var logger, isThrottled, now, timeSinceLastComplete, delay_1, _a, next, preview, nextPreviewPresentationId;
             var _this = this;
             return __generator(this, function (_b) {
                 switch (_b.label) {
@@ -137,12 +138,12 @@ var PlaylistWindowManager = /** @class */ (function (_super) {
                                 isThrottled = true;
                                 // Throttle presentation complete handler if we have not throttled already.
                                 if (this.currentPresentationCompleteTimeout === null) {
-                                    delay = minTimeBetweenCompletes - timeSinceLastComplete + 1;
-                                    logger.warn("Throttling presentation complete handler for ".concat(delay, "ms because it was called again in less than ").concat(minTimeBetweenCompletes, "ms"));
+                                    delay_1 = minTimeBetweenCompletes - timeSinceLastComplete + 1;
+                                    logger.warn("Throttling presentation complete handler for ".concat(delay_1, "ms because it was called again in less than ").concat(minTimeBetweenCompletes, "ms"));
                                     this.currentPresentationCompleteTimeout = setTimeout(function () {
                                         _this.currentPresentationCompleteTimeout = null;
                                         _this.handleCurrentPresentationComplete();
-                                    }, delay);
+                                    }, delay_1);
                                 }
                                 else {
                                     logger.warn("Skipping presentation complete because it is currently throttled");
@@ -940,13 +941,19 @@ var PlaylistWindowManager = /** @class */ (function (_super) {
                         if (this.nextWindow) {
                             this.nextWindow.show(this.isFocused);
                         }
-                        if (!this.currentWindow) return [3 /*break*/, 2];
-                        this.currentWindow.hide();
-                        return [4 /*yield*/, this.removeCurrentWindow()];
+                        if (!this.currentWindow) return [3 /*break*/, 4];
+                        if (!(this.options.closeWindowDelayMs !== undefined)) return [3 /*break*/, 2];
+                        return [4 /*yield*/, delay(this.options.closeWindowDelayMs)];
                     case 1:
                         _a.sent();
                         _a.label = 2;
                     case 2:
+                        this.currentWindow.hide();
+                        return [4 /*yield*/, this.removeCurrentWindow()];
+                    case 3:
+                        _a.sent();
+                        _a.label = 4;
+                    case 4:
                         // Swap windows.
                         this.currentWindow = this.nextWindow;
                         if (this.currentWindow) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raydiant/playlist-window-manager",
-  "version": "0.28.1",
+  "version": "0.28.2-rc2",
   "description": "Window manager for playlist playback.",
   "main": "build/index.js",
   "scripts": {

package/src/BrowserWindowRenderer.ts

@@ -114,6 +114,7 @@ export default class BrowserWindowRenderer
               },
             })
           : this.createOrGetPooledBrowserWindow(windowId, {
+              paintWhenInitiallyHidden: false,
               show: false,
               /**
                * This option fixes window not being able to take fullscreen leaving

package/src/PlaylistWindowManager.ts

@@ -70,6 +70,8 @@ export interface PlaylistWindowManagerOptions {
   // Set to true to render the presentation window as a preview.
   isPreview?: boolean;
 
+  closeWindowDelayMs?: number;
+
   // Called when the current window wants to enable the interrupt lock
   onEnableInterruptLock?: () => void;
 
@@ -87,6 +89,8 @@ export interface PlaylistWindowManagerOptions {
 // to prevent log spam causing devices to become unresponsive.
 const minTimeBetweenCompletes = 3000;
 
+const delay = (ms: number) => new Promise((res) => setTimeout(res, ms));
+
 interface EvalRuleEngineResult {
   next: RE.NextPresentationID;
   preview: RE.NextPresentationID;
@@ -750,6 +754,13 @@ export default class PlaylistWindowManager extends EventEmitter {
     // If current window isn't defined we don't need to hide it but we still
     // want to make the next window the new current window.
     if (this.currentWindow) {
+      // TODO: Kind of a hack, could use win.on('show') event to trigger hide?
+      // Would only work in electron environment so need to do in BrowserWindowRenderer.
+      // Should show be async then?
+      // Or try paintWhenInitiallyHidden: false?
+      if (this.options.closeWindowDelayMs !== undefined) {
+        await delay(this.options.closeWindowDelayMs);
+      }
       this.currentWindow.hide();
       await this.removeCurrentWindow();
     }