Moving cloneTemplate to abr-menu

brrd · Apr 26, 2016 · bca016d · bca016d
1 parent a642a9e
commit bca016d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/abr-menu.js b/app/abr-menu.js
@@ -160,7 +160,8 @@ function exportHtmlMenuGenerator (submenu, config) {
 // Electron's menu wrapper
 function AbrMenu (abrApp, abrWin, menuTemplate, config) {
     this.abrWin = abrWin;
-    var preprocessedMenuTemplate = preprocessTemplate(abrApp, menuTemplate, config);
+    var cloneTemplate = JSON.parse(JSON.stringify(menuTemplate)); // Electron modifies the template while building the menu so we need to clone it before
+    var preprocessedMenuTemplate = preprocessTemplate(abrApp, cloneTemplate, config);
     this.menu = Menu.buildFromTemplate(preprocessedMenuTemplate);
 }
 

diff --git a/app/abr-window.js b/app/abr-window.js
@@ -50,8 +50,7 @@ function AbrWindow (abrApp, path) {
         }
     }
     this.config = createConfig();
-    var cloneTemplate = JSON.parse(JSON.stringify(menuTemplate)); // Electron modifies the template while building the menu so we need to clone it before
-    this.menu = new AbrMenu(abrApp, this, cloneTemplate, this.config);
+    this.menu = new AbrMenu(abrApp, this, menuTemplate, this.config);
     // Context
     this.contextMenu = new AbrMenu(abrApp, this, contextMenuTemplate, this.config);
     this.open();