Bug 1395194 - Copy URL confirmation should fade out at timeout. r?adw draft
authorJared Wein <jwein@mozilla.com>
Wed, 30 Aug 2017 16:23:05 -0400
changeset 656167 8e618177580094c2f655ebe1689539d0e9aaa137
parent 655774 ab2d700fda2b4934d24227216972dce9fac19b74
child 729041 8c3ba56d0d3e1c667fa5f708867754df52978758
push id77100
push userbmo:jaws@mozilla.com
push dateWed, 30 Aug 2017 20:23:22 +0000
reviewersadw
bugs1395194
milestone57.0a1
Bug 1395194 - Copy URL confirmation should fade out at timeout. r?adw MozReview-Commit-ID: DTrzSOaft3N
browser/base/content/browser-pageActions.js
--- a/browser/base/content/browser-pageActions.js
+++ b/browser/base/content/browser-pageActions.js
@@ -716,17 +716,17 @@ var BrowserPageActionFeedback = {
     }, {once: true});
     this.panelNode.addEventListener("popuphidden", () => {
       this.feedbackAnimationBox.removeAttribute("animate");
     }, {once: true});
 
     // The timeout value used here allows the panel to stay open for
     // 1 second after the text transition (duration=120ms) has finished.
     setTimeout(() => {
-      this.panelNode.hidePopup();
+      this.panelNode.hidePopup(true);
     }, Services.prefs.getIntPref("browser.pageActions.feedbackTimeoutMS", 1120));
   },
 };
 
 // built-in actions below //////////////////////////////////////////////////////
 
 // bookmark
 BrowserPageActions.bookmark = {