From 81fed8a1d8df69eab6f900d97e608ce4df610dd5 Mon Sep 17 00:00:00 2001 From: "mr. M" Date: Fri, 21 Feb 2025 19:54:21 +0100 Subject: [PATCH] Refactor event listener for ZenThemePicker panel to improve readability and maintainability --- src/browser/base/zen-components/ZenWelcome.mjs | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/src/browser/base/zen-components/ZenWelcome.mjs b/src/browser/base/zen-components/ZenWelcome.mjs index 38a0ea511..730296af6 100644 --- a/src/browser/base/zen-components/ZenWelcome.mjs +++ b/src/browser/base/zen-components/ZenWelcome.mjs @@ -435,12 +435,16 @@ document.getElementById('zen-welcome-page-content').appendChild(anchor); gZenThemePicker.panel.setAttribute('noautohide', 'true'); gZenThemePicker.panel.setAttribute('consumeoutsideclicks', 'false'); - gZenThemePicker.panel.addEventListener('popupshowing', () => { - const panelRect = gZenThemePicker.panel.getBoundingClientRect(); - // 20 is the shadow width * 2 - anchor.style.height = panelRect.height - 20 + 'px'; - anchor.style.width = panelRect.width - 20 + 'px'; - }, { once: true }); + gZenThemePicker.panel.addEventListener( + 'popupshowing', + () => { + const panelRect = gZenThemePicker.panel.getBoundingClientRect(); + // 20 is the shadow width * 2 + anchor.style.height = panelRect.height - 20 + 'px'; + anchor.style.width = panelRect.width - 20 + 'px'; + }, + { once: true } + ); PanelMultiView.openPopup(gZenThemePicker.panel, anchor, { position: 'overlap', }); @@ -449,7 +453,7 @@ async fadeOut() { gZenThemePicker.panel.removeAttribute('noautohide'); gZenThemePicker.panel.removeAttribute('consumeoutsideclicks'); - if (AppConstants.platform != "macosx") { + if (AppConstants.platform != 'macosx') { await animate(gZenThemePicker.panel, { opacity: [1, 0] }); } gZenThemePicker.panel.hidePopup();