@raydiant/playlist-window-manager 0.0.0-test-7 → 0.0.0-test-9

package/build/BrowserWindowRenderer.d.ts

@@ -33,6 +33,13 @@ export default class BrowserWindowRenderer implements PresentationWindowRenderer
     sendMessageObject(windowId: string, type: string, payload?: any): void;
     muteWindow(windowId: string): void;
     unMuteWindow(windowId: string): void;
+    /**
+     * This function is designed to solve missing 1px issue.
+     * If we set 1920x1080 bounds on 1920x1080 display
+     * then the actual BrowserWindow resolution would be
+     * 1919x1079.
+     */
+    private setWindowOrViewBounds;
     private getWindow;
     private moveWindowOffscreen;
     private addWindowToParent;

package/build/BrowserWindowRenderer.js

@@ -76,6 +76,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 var path = __importStar(require("path"));
 var electron_1 = require("electron");
 var WindowUnresponsiveError_1 = __importDefault(require("./WindowUnresponsiveError"));
+var windowUtils_1 = require("./windowUtils");
 var BrowserWindowRenderer = /** @class */ (function () {
     function BrowserWindowRenderer(opts) {
         this.opts = opts;
@@ -88,15 +89,14 @@ var BrowserWindowRenderer = /** @class */ (function () {
             show: false,
             frame: this.opts.frame,
             hasShadow: this.opts.hasShadow,
+            fullscreen: (0, windowUtils_1.isFullscreen)(this.opts.bounds),
             webPreferences: {
                 preload: path.join(__dirname, 'preload.js'),
             },
         });
         this.proxySettings = null;
         opts.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts constructor1 ".concat(JSON.stringify(opts.bounds)));
-        // TODO zlv added
-        this.offscreenWindow.setFullScreen(true);
-        this.offscreenWindow.setBounds(opts.bounds);
+        this.setWindowOrViewBounds(this.offscreenWindow, opts.bounds);
         opts.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts constructor2 ".concat(JSON.stringify({
             bounds: this.offscreenWindow.getBounds(),
             normalBounds: this.offscreenWindow.getNormalBounds(),
@@ -188,16 +188,7 @@ var BrowserWindowRenderer = /** @class */ (function () {
                             window.view.setBackgroundColor(backgroundColor === 'transparent' ? '#00000000' : backgroundColor);
                         }
                         this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts createWindow4: ".concat(JSON.stringify(window.bounds)));
-                        // TODO zlv added
-                        if (window.view instanceof electron_1.BrowserWindow) {
-                            this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts createWindow5: setFullScreen");
-                            window.view.setFullScreen(true);
-                            window.view.setBounds(window.bounds);
-                        }
-                        else {
-                            this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts createWindow6: setBounds");
-                            window.view.setBounds(window.bounds);
-                        }
+                        this.setWindowOrViewBounds(window.view, window.bounds);
                         this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts createWindow7: ".concat(JSON.stringify(window.view.getBounds())));
                         if (!this.proxySettings) return [3 /*break*/, 2];
                         return [4 /*yield*/, window.view.webContents.session.setProxy(this.proxySettings)];
@@ -239,12 +230,11 @@ var BrowserWindowRenderer = /** @class */ (function () {
         this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds1: ".concat(JSON.stringify(bounds)));
         window.bounds = bounds;
         if (window.view instanceof electron_1.BrowserWindow && window.parentWindowId) {
-            this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds2: ".concat(JSON.stringify(__assign(__assign({}, window.bounds), { x: window.bounds.x + this.opts.bounds.x, y: window.bounds.y + this.opts.bounds.y }))));
             // Child BrowserWindows are absolutely positioned so we need to add the main windows
             // x/y to when updating the child window's bounds.
-            // TODO zlv added
-            window.view.setFullScreen(true);
-            window.view.setBounds(__assign(__assign({}, window.bounds), { x: window.bounds.x + this.opts.bounds.x, y: window.bounds.y + this.opts.bounds.y }));
+            var newBounds = __assign(__assign({}, window.bounds), { x: window.bounds.x + this.opts.bounds.x, y: window.bounds.y + this.opts.bounds.y });
+            this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds2: ".concat(JSON.stringify(newBounds)));
+            this.setWindowOrViewBounds(window.view, newBounds);
             this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds3: ".concat(JSON.stringify({
                 bounds: window.view.getBounds(),
                 normalBounds: window.view.getNormalBounds(),
@@ -253,7 +243,7 @@ var BrowserWindowRenderer = /** @class */ (function () {
         }
         else {
             this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds4: ".concat(JSON.stringify(window.bounds)));
-            window.view.setBounds(window.bounds);
+            this.setWindowOrViewBounds(window.view, window.bounds);
             this.logger.info("playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds5: ".concat(JSON.stringify(window.view.getBounds())));
         }
     };
@@ -358,6 +348,18 @@ var BrowserWindowRenderer = /** @class */ (function () {
         }
         window.view.webContents.setAudioMuted(false);
     };
+    /**
+     * This function is designed to solve missing 1px issue.
+     * If we set 1920x1080 bounds on 1920x1080 display
+     * then the actual BrowserWindow resolution would be
+     * 1919x1079.
+     */
+    BrowserWindowRenderer.prototype.setWindowOrViewBounds = function (window, bounds) {
+        window.setBounds(bounds);
+        if (window instanceof electron_1.BrowserWindow) {
+            window.setFullScreen((0, windowUtils_1.isFullscreen)(bounds));
+        }
+    };
     BrowserWindowRenderer.prototype.getWindow = function (windowId) {
         return this.windows[windowId];
     };

package/build/windowUtils.d.ts

@@ -0,0 +1,2 @@
+import { WindowBounds } from './types';
+export declare const isFullscreen: (bounds: WindowBounds) => boolean;

package/build/windowUtils.js

@@ -0,0 +1,9 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.isFullscreen = void 0;
+var electron_1 = require("electron");
+var isFullscreen = function (bounds) {
+    var screenBounds = electron_1.screen.getPrimaryDisplay().bounds;
+    return (bounds.width === screenBounds.width && bounds.height === screenBounds.height);
+};
+exports.isFullscreen = isFullscreen;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raydiant/playlist-window-manager",
-  "version": "0.0.0-test-7",
+  "version": "0.0.0-test-9",
   "description": "Window manager for playlist playback.",
   "main": "build/index.js",
   "scripts": {

package/src/BrowserWindowRenderer.ts

@@ -16,6 +16,7 @@ import {
   Logger,
 } from './types';
 import WindowUnresponsiveError from './WindowUnresponsiveError';
+import { isFullscreen } from './windowUtils';
 
 interface ProxySettings {
   proxyRules: string;
@@ -62,6 +63,7 @@ export default class BrowserWindowRenderer
     show: false,
     frame: this.opts.frame,
     hasShadow: this.opts.hasShadow,
+    fullscreen: isFullscreen(this.opts.bounds),
     webPreferences: {
       preload: path.join(__dirname, 'preload.js'),
     },
@@ -77,9 +79,7 @@ export default class BrowserWindowRenderer
         opts.bounds
       )}`
     );
-    // TODO zlv added
-    this.offscreenWindow.setFullScreen(true);
-    this.offscreenWindow.setBounds(opts.bounds);
+    this.setWindowOrViewBounds(this.offscreenWindow, opts.bounds);
     opts.logger.info(
       `playlist-window-manager/src/BrowserWindowRenderer.ts constructor2 ${JSON.stringify(
         {
@@ -233,19 +233,7 @@ export default class BrowserWindowRenderer
         window.bounds
       )}`
     );
-    // TODO zlv added
-    if (window.view instanceof BrowserWindow) {
-      this.logger.info(
-        `playlist-window-manager/src/BrowserWindowRenderer.ts createWindow5: setFullScreen`
-      );
-      window.view.setFullScreen(true);
-      window.view.setBounds(window.bounds);
-    } else {
-      this.logger.info(
-        `playlist-window-manager/src/BrowserWindowRenderer.ts createWindow6: setBounds`
-      );
-      window.view.setBounds(window.bounds);
-    }
+    this.setWindowOrViewBounds(window.view, window.bounds);
     this.logger.info(
       `playlist-window-manager/src/BrowserWindowRenderer.ts createWindow7: ${JSON.stringify(
         window.view.getBounds()
@@ -287,25 +275,19 @@ export default class BrowserWindowRenderer
     window.bounds = bounds;
 
     if (window.view instanceof BrowserWindow && window.parentWindowId) {
-      this.logger.info(
-        `playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds2: ${JSON.stringify(
-          {
-            ...window.bounds,
-            x: window.bounds.x + this.opts.bounds.x,
-            y: window.bounds.y + this.opts.bounds.y,
-          }
-        )}`
-      );
-
       // Child BrowserWindows are absolutely positioned so we need to add the main windows
       // x/y to when updating the child window's bounds.
-      // TODO zlv added
-      window.view.setFullScreen(true);
-      window.view.setBounds({
+      const newBounds = {
         ...window.bounds,
         x: window.bounds.x + this.opts.bounds.x,
         y: window.bounds.y + this.opts.bounds.y,
-      });
+      };
+      this.logger.info(
+        `playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds2: ${JSON.stringify(
+          newBounds
+        )}`
+      );
+      this.setWindowOrViewBounds(window.view, newBounds);
 
       this.logger.info(
         `playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds3: ${JSON.stringify(
@@ -322,7 +304,7 @@ export default class BrowserWindowRenderer
           window.bounds
         )}`
       );
-      window.view.setBounds(window.bounds);
+      this.setWindowOrViewBounds(window.view, window.bounds);
       this.logger.info(
         `playlist-window-manager/src/BrowserWindowRenderer.ts updateWindowBounds5: ${JSON.stringify(
           window.view.getBounds()
@@ -446,6 +428,22 @@ export default class BrowserWindowRenderer
     window.view.webContents.setAudioMuted(false);
   }
 
+  /**
+   * This function is designed to solve missing 1px issue.
+   * If we set 1920x1080 bounds on 1920x1080 display
+   * then the actual BrowserWindow resolution would be
+   * 1919x1079.
+   */
+  private setWindowOrViewBounds(
+    window: BrowserWindow | BrowserView,
+    bounds: WindowBounds
+  ) {
+    window.setBounds(bounds);
+    if (window instanceof BrowserWindow) {
+      window.setFullScreen(isFullscreen(bounds));
+    }
+  }
+
   private getWindow(windowId: string): BrowserWindowRendererWindow | null {
     return this.windows[windowId];
   }

package/src/windowUtils.ts

@@ -0,0 +1,9 @@
+import { screen } from 'electron';
+import { WindowBounds } from './types';
+
+export const isFullscreen = (bounds: WindowBounds): boolean => {
+  const screenBounds = screen.getPrimaryDisplay().bounds;
+  return (
+    bounds.width === screenBounds.width && bounds.height === screenBounds.height
+  );
+};