@raydiant/playlist-window-manager 0.27.3-rc.1 → 0.28.0

package/build/BrowserViewRenderer.d.ts

@@ -13,6 +13,7 @@ export default class BrowserViewRenderer implements PresentationWindowRenderer {
     removeWindow(windowId: string): Promise<void>;
     sendMessageString(windowId: string, message: string): void;
     sendMessageObject(windowId: string, type: string, payload?: any): void;
+    sendOnlineStatus(windowId: string, isOnline: boolean): void;
     muteWindow(windowId: string): void;
     unMuteWindow(windowId: string): void;
     private getWindow;

package/build/BrowserViewRenderer.js

@@ -191,6 +191,13 @@ var BrowserViewRenderer = /** @class */ (function () {
         }
         window.view.webContents.send('electron-main', { type: type, payload: payload });
     };
+    BrowserViewRenderer.prototype.sendOnlineStatus = function (windowId, isOnline) {
+        var window = this.getWindow(windowId);
+        if (!window) {
+            throw new Error("Failed to send isOnline status to window, missing window id '".concat(windowId, "'"));
+        }
+        window.view.webContents.send('online-status', isOnline);
+    };
     BrowserViewRenderer.prototype.muteWindow = function (windowId) {
         var window = this.getWindow(windowId);
         if (!window) {

package/build/BrowserWindowRenderer.js

@@ -90,7 +90,7 @@ var BrowserWindowRenderer = /** @class */ (function () {
              * This option fixes window not being able to take fullscreen leaving
              * 1px gap at the right and bottom of the screen.
              */
-            // resizable: false,
+            resizable: false,
             frame: this.opts.frame,
             hasShadow: this.opts.hasShadow,
             webPreferences: {
@@ -133,7 +133,7 @@ var BrowserWindowRenderer = /** @class */ (function () {
                                      * This option fixes window not being able to take fullscreen leaving
                                      * 1px gap at the right and bottom of the screen.
                                      */
-                                    // resizable: false,
+                                    resizable: false,
                                     frame: this.opts.frame,
                                     // NOTE: useWindowPool: true does not work with renderChildWindowsAsBrowserViews: false
                                     // because the parent can't be changed after creating the BrowserWindow.

package/build/PresentationWindow.js

@@ -757,11 +757,13 @@ var PresentationWindow = /** @class */ (function () {
     };
     PresentationWindow.prototype.sendOnlineStatus = function (isOnline) {
         var _a = this.options, windowId = _a.windowId, renderer = _a.renderer;
-        // Send online-status event. This Separate from sendMessageObject since we
-        // are bypassing Raydiant SDK message receivers.
+        // Send online-status event. We are not using sendMessageObject since we
+        // are bypassing Raydiant SDK's message receivers and communicating directly
+        // with the preload script.
         if (renderer.sendOnlineStatus) {
             renderer.sendOnlineStatus(windowId, isOnline);
         }
+        // Forward the event to the child window managers.
         Object.values(this.windowManagers).forEach(function (wm) {
             wm.sendOnlineStatus(isOnline);
         });

package/build/preload.js

@@ -35,13 +35,10 @@ function makeWindowEvent(type) {
 var isOnline = electron_1.ipcRenderer.sendSync('get-online-status');
 // Listen for online status changes from the main process.
 electron_1.ipcRenderer.on('online-status', function (_, nextIsOnline) {
-    console.log("Online status: ".concat(nextIsOnline));
     if (nextIsOnline !== isOnline) {
-        console.log('Online status changed, updating navigator.onLine');
         isOnline = nextIsOnline;
         var events_1 = makeWindowEvent(isOnline ? 'online' : 'offline');
         var listeners = isOnline ? onlineListeners : offlineListeners;
-        console.log("Running ".concat(isOnline ? 'online' : 'offline', " listeners, count: ").concat(listeners.size));
         listeners.forEach(function (listener) {
             if (typeof listener === 'function') {
                 listener(events_1);
@@ -57,21 +54,19 @@ electron_1.ipcRenderer.on('online-status', function (_, nextIsOnline) {
 // when running on the device.
 Object.defineProperty(navigator, 'onLine', {
     get: function () {
-        console.log("navigator.onLine called, returning ".concat(isOnline));
         return isOnline;
     },
 });
 var offlineListeners = new Set();
 var onlineListeners = new Set();
+// Intercept online/offline add event listeners so we can call them when online status changes.
 var originalAddEventListener = window.addEventListener.bind(window);
 Object.defineProperty(window, 'addEventListener', {
     value: function (type, listener) {
         if (type === 'online') {
-            console.log("Intercepted online addEventListener");
             onlineListeners.add(listener);
         }
         else if (type === 'offline') {
-            console.log("Intercepted offline addEventListener");
             offlineListeners.add(listener);
         }
         else {
@@ -79,15 +74,14 @@ Object.defineProperty(window, 'addEventListener', {
         }
     },
 });
+// Intercept online/offline remove event listeners so we can remove our custom listeners.
 var originalRemoveEventListener = window.removeEventListener.bind(window);
 Object.defineProperty(window, 'removeEventListener', {
     value: function (type, listener) {
         if (type === 'online') {
-            console.log("Intercepted online removeEventListener");
             onlineListeners.delete(listener);
         }
         else if (type === 'offline') {
-            console.log("Intercepted offline removeEventListener");
             offlineListeners.delete(listener);
         }
         else {
@@ -95,27 +89,3 @@ Object.defineProperty(window, 'removeEventListener', {
         }
     },
 });
-// // TODO: Remove this (for local dev only).
-// (window as any).simulateOnline = () => {
-//   isOnline = true;
-//   console.log(`Running online listeners, count: ${onlineListeners.size}`);
-//   onlineListeners.forEach((listener) => {
-//     if (typeof listener === 'function') {
-//       listener(makeWindowEvent('online'));
-//     } else if (listener.handleEvent) {
-//       listener.handleEvent(makeWindowEvent('online'));
-//     }
-//   });
-// };
-// // TODO: Remove this (for local dev only).
-// (window as any).simulateOffline = () => {
-//   isOnline = false;
-//   console.log(`Running offline listeners, count: ${offlineListeners.size}`);
-//   offlineListeners.forEach((listener) => {
-//     if (typeof listener === 'function') {
-//       listener(makeWindowEvent('offline'));
-//     } else if (listener.handleEvent) {
-//       listener.handleEvent(makeWindowEvent('offline'));
-//     }
-//   });
-// };

package/package.json

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

package/src/BrowserViewRenderer.ts

@@ -154,6 +154,17 @@ export default class BrowserViewRenderer implements PresentationWindowRenderer {
     window.view.webContents.send('electron-main', { type, payload });
   }
 
+  public sendOnlineStatus(windowId: string, isOnline: boolean) {
+    const window = this.getWindow(windowId);
+    if (!window) {
+      throw new Error(
+        `Failed to send isOnline status to window, missing window id '${windowId}'`
+      );
+    }
+
+    window.view.webContents.send('online-status', isOnline);
+  }
+
   public muteWindow(windowId: string) {
     const window = this.getWindow(windowId);
     if (!window) {

package/src/BrowserWindowRenderer.ts

@@ -67,7 +67,7 @@ export default class BrowserWindowRenderer
      * This option fixes window not being able to take fullscreen leaving
      * 1px gap at the right and bottom of the screen.
      */
-    // resizable: false,
+    resizable: false,
     frame: this.opts.frame,
     hasShadow: this.opts.hasShadow,
     webPreferences: {
@@ -119,7 +119,7 @@ export default class BrowserWindowRenderer
                * This option fixes window not being able to take fullscreen leaving
                * 1px gap at the right and bottom of the screen.
                */
-              // resizable: false,
+              resizable: false,
               frame: this.opts.frame,
               // NOTE: useWindowPool: true does not work with renderChildWindowsAsBrowserViews: false
               // because the parent can't be changed after creating the BrowserWindow.

package/src/PresentationWindow.ts

@@ -528,13 +528,15 @@ export default class PresentationWindow {
 
   public sendOnlineStatus(isOnline: boolean) {
     const { windowId, renderer } = this.options;
-    // Send online-status event. This Separate from sendMessageObject since we
-    // are bypassing Raydiant SDK message receivers.
 
+    // Send online-status event. We are not using sendMessageObject since we
+    // are bypassing Raydiant SDK's message receivers and communicating directly
+    // with the preload script.
     if (renderer.sendOnlineStatus) {
       renderer.sendOnlineStatus(windowId, isOnline);
     }
 
+    // Forward the event to the child window managers.
     Object.values(this.windowManagers).forEach((wm) => {
       wm.sendOnlineStatus(isOnline);
     });

package/src/preload.ts

@@ -50,21 +50,12 @@ let isOnline = ipcRenderer.sendSync('get-online-status');
 ipcRenderer.on(
   'online-status',
   (_: IpcRendererEvent, nextIsOnline: boolean) => {
-    console.log(`Online status: ${nextIsOnline}`);
-
     if (nextIsOnline !== isOnline) {
-      console.log('Online status changed, updating navigator.onLine');
       isOnline = nextIsOnline;
 
       const events = makeWindowEvent(isOnline ? 'online' : 'offline');
       const listeners = isOnline ? onlineListeners : offlineListeners;
 
-      console.log(
-        `Running ${isOnline ? 'online' : 'offline'} listeners, count: ${
-          listeners.size
-        }`
-      );
-
       listeners.forEach((listener) => {
         if (typeof listener === 'function') {
           listener(events);
@@ -81,7 +72,6 @@ ipcRenderer.on(
 // when running on the device.
 Object.defineProperty(navigator, 'onLine', {
   get: () => {
-    console.log(`navigator.onLine called, returning ${isOnline}`);
     return isOnline;
   },
 });
@@ -89,14 +79,13 @@ Object.defineProperty(navigator, 'onLine', {
 const offlineListeners = new Set<EventListenerOrEventListenerObject>();
 const onlineListeners = new Set<EventListenerOrEventListenerObject>();
 
+// Intercept online/offline add event listeners so we can call them when online status changes.
 const originalAddEventListener = window.addEventListener.bind(window);
 Object.defineProperty(window, 'addEventListener', {
   value: (type: string, listener: EventListenerOrEventListenerObject) => {
     if (type === 'online') {
-      console.log(`Intercepted online addEventListener`);
       onlineListeners.add(listener);
     } else if (type === 'offline') {
-      console.log(`Intercepted offline addEventListener`);
       offlineListeners.add(listener);
     } else {
       return originalAddEventListener(type, listener);
@@ -104,45 +93,16 @@ Object.defineProperty(window, 'addEventListener', {
   },
 });
 
+// Intercept online/offline remove event listeners so we can remove our custom listeners.
 const originalRemoveEventListener = window.removeEventListener.bind(window);
 Object.defineProperty(window, 'removeEventListener', {
   value: (type: string, listener: EventListenerOrEventListenerObject) => {
     if (type === 'online') {
-      console.log(`Intercepted online removeEventListener`);
       onlineListeners.delete(listener);
     } else if (type === 'offline') {
-      console.log(`Intercepted offline removeEventListener`);
       offlineListeners.delete(listener);
     } else {
       return originalRemoveEventListener(type, listener);
     }
   },
 });
-
-// // TODO: Remove this (for local dev only).
-// (window as any).simulateOnline = () => {
-//   isOnline = true;
-
-//   console.log(`Running online listeners, count: ${onlineListeners.size}`);
-//   onlineListeners.forEach((listener) => {
-//     if (typeof listener === 'function') {
-//       listener(makeWindowEvent('online'));
-//     } else if (listener.handleEvent) {
-//       listener.handleEvent(makeWindowEvent('online'));
-//     }
-//   });
-// };
-
-// // TODO: Remove this (for local dev only).
-// (window as any).simulateOffline = () => {
-//   isOnline = false;
-
-//   console.log(`Running offline listeners, count: ${offlineListeners.size}`);
-//   offlineListeners.forEach((listener) => {
-//     if (typeof listener === 'function') {
-//       listener(makeWindowEvent('offline'));
-//     } else if (listener.handleEvent) {
-//       listener.handleEvent(makeWindowEvent('offline'));
-//     }
-//   });
-// };