smueller/youtube-music
1
0
Fork 0
mirror of https://github.com/th-ch/youtube-music.git synced 2026-01-11 18:41:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix memory leak +

Browse Source 
in-app-menu updates menu only if needed
This commit is contained in:
Araxeus
2021-04-03 21:53:39 +03:00
parent 061e4a9e5f
commit 317521bba6
2 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
index.js
View File

@ -152,7 +152,13 @@ function createMainWindow() {
return win;
}
let createdWindow = false;
app.on("browser-window-created", (event, win) => {
//Ensure listeners aren't registered when creating input dialog
if(createdWindow){
return;
}
createdWindow = true;
loadPlugins(win);
win.webContents.on("did-fail-load", () => {

16
plugins/in-app-menu/back.js
View File

@ -16,7 +16,7 @@ const originalBuildMenu = Menu.buildFromTemplate;
// This function natively gets called on all submenu so no more reason to use recursion
Menu.buildFromTemplate = (template) => {
// Fix checkboxes and radio buttons
updateCheckboxesAndRadioButtons(template);
updateTemplate(template);
// return as normal
return originalBuildMenu(template);
@ -55,22 +55,26 @@ function switchMenuVisibility() {
win.webContents.send("updateMenu", visible);
}
function checkCheckbox(item) {
//check item
function updateCheckboxesAndRadioButtons(item, isRadio, hasSubmenu) {
if (!isRadio) {
//fix checkbox
item.checked = !item.checked;
//update menu (closes it)
}
//update menu if radio / hasSubmenu
if (isRadio || hasSubmenu) {
win.webContents.send("updateMenu", true);
}
}
// Update checkboxes/radio buttons
function updateCheckboxesAndRadioButtons(template) {
function updateTemplate(template) {
for (let item of template) {
// Change onClick of checkbox+radio
if ((item.type === "checkbox" || item.type === "radio") && !item.fixed) {
let originalOnclick = item.click;
item.click = (itemClicked) => {
originalOnclick(itemClicked);
checkCheckbox(itemClicked);
updateCheckboxesAndRadioButtons(itemClicked, item.type==='radio', item.hasSubmenu);
};
item.fixed = true;
}