Rename variables and function for clarity

This commit is contained in:
Araxeus
2021-03-30 02:00:35 +03:00
parent b67b1bed13
commit ae7def2313

View File

@ -11,7 +11,7 @@ mainMenuTemplate = function (winHook) {
//get template //get template
let template = originTemplate(winHook); let template = originTemplate(winHook);
//fix checkbox and roles //fix checkbox and roles
fixCheck(template); fixMenu(template);
//return as normal //return as normal
return template; return template;
} }
@ -52,12 +52,12 @@ function switchMenuVisibility() {
} }
//go over each item in menu //go over each item in menu
function fixCheck(ogTemplate) { function fixMenu(template) {
for (let position in ogTemplate) { for (let index in template) {
let item = ogTemplate[position]; let item = template[index];
//apply function on submenu //apply function on submenu
if (item.submenu != null) { if (item.submenu != null) {
fixCheck(item.submenu); fixMenu(item.submenu);
} }
//change onClick of checkbox+radio //change onClick of checkbox+radio
else if (item.type === 'checkbox' || item.type === 'radio') { else if (item.type === 'checkbox' || item.type === 'radio') {
@ -98,7 +98,7 @@ function fixRoles(MenuItem) {
MenuItem.click = () => { win.webContents.setZoomLevel(0); } MenuItem.click = () => { win.webContents.setZoomLevel(0); }
break; break;
default: default:
console.log(MenuItem.role + ' was not expected'); console.log(`Error fixing MenuRoles: "${MenuItem.role}" was not expected`);
} }
delete MenuItem.role; delete MenuItem.role;
} }