Added TODOs related to opening child window for screensharing/video calls.

1 files changed, 5 insertions, 0 deletions

diff --git a/recipes/skype/webview.js b/recipes/skype/webview.js
index f8b4d7b..c20726c 100644
--- a/recipes/skype/webview.js
+++ b/recipes/skype/webview.js
@@ -35,12 +35,17 @@ module.exports = (Franz, settings) => {
       if (url.includes('views/imgpsh_fullsize_anim')) {
         event.preventDefault();
         event.stopPropagation();
+        // TODO: Can we send an ipc event 'open-browser-window' to open the child window? (see the slack recipe for how to send an ipc message)
+        // TODO: Can we change the slack recipe to add a clickHandler for screensharing/video calls? (https://github.com/getferdi/ferdi/issues/1697)
         const win = new BrowserWindow({
           width: 800,
           height: window.innerHeight,
           minWidth: 600,
           webPreferences: {
             partition: `persist:service-${settings.id}`,
+            // TODO: Aren't these needed here?
+            // contextIsolation: false,
+            // enableRemoteModule: true,
           },
         });
         win.loadURL(url);