From 6b12eed38b22a3a4cc3e2bb75b6faa4cc956c38a Mon Sep 17 00:00:00 2001 From: Antonio Enriquez <60269998+aenriq@users.noreply.github.com> Date: Wed, 23 Jul 2025 10:19:11 -0700 Subject: [PATCH] fix: Fix copy button not working in calculator and unit conversion results (#9588) Resolves race condition where urlbar view close was interrupting engagement events, preventing onEngagement from being called. --- src/zen/common/ZenUIManager.mjs | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/src/zen/common/ZenUIManager.mjs b/src/zen/common/ZenUIManager.mjs index 7372377cd..8173287d4 100644 --- a/src/zen/common/ZenUIManager.mjs +++ b/src/zen/common/ZenUIManager.mjs @@ -398,17 +398,19 @@ var gZenUIManager = { } if (gURLBar.focused) { - gURLBar.view.close({ elementPicked: onSwitch }); - gURLBar.updateTextOverflow(); + setTimeout(() => { + gURLBar.view.close({ elementPicked: onSwitch }); + gURLBar.updateTextOverflow(); - // Ensure tab and browser are valid before updating state - const selectedTab = gBrowser.selectedTab; - if (selectedTab && selectedTab.linkedBrowser && !selectedTab.closing && onSwitch) { - const browserState = gURLBar.getBrowserState(selectedTab.linkedBrowser); - if (browserState) { - browserState.urlbarFocused = false; + // Ensure tab and browser are valid before updating state + const selectedTab = gBrowser.selectedTab; + if (selectedTab && selectedTab.linkedBrowser && !selectedTab.closing && onSwitch) { + const browserState = gURLBar.getBrowserState(selectedTab.linkedBrowser); + if (browserState) { + browserState.urlbarFocused = false; + } } - } + }, 0); } }); },