From 104650627d0dea207b5803f89d648a4363d6af8d Mon Sep 17 00:00:00 2001 From: toasted-nutbread Date: Sun, 26 Jul 2020 16:53:24 -0400 Subject: [PATCH] Merge redundant toggle handling code (#693) --- ext/bg/js/context-main.js | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/ext/bg/js/context-main.js b/ext/bg/js/context-main.js index 14013d3f..72abe554 100644 --- a/ext/bg/js/context-main.js +++ b/ext/bg/js/context-main.js @@ -61,13 +61,12 @@ async function setupOptions() { }; const options = await api.optionsGet(optionsContext); - const toggle = document.querySelector('#enable-search'); - toggle.checked = options.general.enable; - toggle.addEventListener('change', () => api.commandExec('toggle'), false); - - const toggle2 = document.querySelector('#enable-search2'); - toggle2.checked = options.general.enable; - toggle2.addEventListener('change', () => api.commandExec('toggle'), false); + const extensionEnabled = options.general.enable; + const onToggleChanged = () => api.commandExec('toggle'); + for (const toggle of document.querySelectorAll('#enable-search,#enable-search2')) { + toggle.checked = extensionEnabled; + toggle.addEventListener('change', onToggleChanged, false); + } setTimeout(() => { document.body.dataset.loaded = 'true';