From d353c13bf1c5c04ca168e23b22e3ebc2c8ab1583 Mon Sep 17 00:00:00 2001
From: David Baker <dave@matrix.org>
Date: Fri, 18 May 2018 11:42:40 +0100
Subject: [PATCH] Comment why we're passing empty objects in

---
 electron_app/src/webcontents-handler.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/electron_app/src/webcontents-handler.js b/electron_app/src/webcontents-handler.js
index 8e079718..cd6833fd 100644
--- a/electron_app/src/webcontents-handler.js
+++ b/electron_app/src/webcontents-handler.js
@@ -61,6 +61,7 @@ function onLinkContextMenu(ev, params) {
             clipboard.writeText(url);
         },
     }));
+    // popup() requires an options object even for no options
     popupMenu.popup({});
     ev.preventDefault();
 }
@@ -88,6 +89,7 @@ function onSelectedContextMenu(ev, params) {
     const items = _CutCopyPasteSelectContextMenus(params);
     const popupMenu = Menu.buildFromTemplate(items);
 
+    // popup() requires an options object even for no options
     popupMenu.popup({});
     ev.preventDefault();
 }
@@ -101,6 +103,7 @@ function onEditableContextMenu(ev, params) {
 
     const popupMenu = Menu.buildFromTemplate(items);
 
+    // popup() requires an options object even for no options
     popupMenu.popup({});
     ev.preventDefault();
 }