Bug 1259108 - Increase timeout of notifications from 12s to 20s to match parity with Chromium. r?mattn draft
authorJared Wein <jwein@mozilla.com>
Wed, 23 Mar 2016 12:54:57 -0400
changeset 343971 0b0856672f6d9f0e4d67712b1abe20fa5688b570
parent 343953 db34f6303717c29e1affdfce92c8f38e6589528b
child 516864 7cde27627a4689877eb2a65e5cb6e5f6d0754ce6
push id13726
push userjwein@mozilla.com
push dateWed, 23 Mar 2016 16:56:13 +0000
reviewersmattn
bugs1259108
milestone48.0a1
Bug 1259108 - Increase timeout of notifications from 12s to 20s to match parity with Chromium. r?mattn MozReview-Commit-ID: 31RjSWuszuC
toolkit/components/alerts/resources/content/alert.css
toolkit/components/alerts/resources/content/alert.js
--- a/toolkit/components/alerts/resources/content/alert.css
+++ b/toolkit/components/alerts/resources/content/alert.css
@@ -1,14 +1,14 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 #alertBox[animate] {
-  animation-duration: 12s;
+  animation-duration: 20s;
   animation-fill-mode: both;
   animation-name: alert-animation;
 }
 
 #alertBox[animate]:not([clicked]):not([closing]):hover {
   animation-play-state: paused;
 }
 
--- a/toolkit/components/alerts/resources/content/alert.js
+++ b/toolkit/components/alerts/resources/content/alert.js
@@ -141,17 +141,17 @@ function prefillAlertInfo() {
         document.getElementById("alertImage").setAttribute("src", window.arguments[0]);
       }
     case 0:
       break;
   }
 }
 
 function onAlertLoad() {
-  const ALERT_DURATION_IMMEDIATE = 12000;
+  const ALERT_DURATION_IMMEDIATE = 20000;
   let alertTextBox = document.getElementById("alertTextBox");
   let alertImageBox = document.getElementById("alertImageBox");
   alertImageBox.style.minHeight = alertTextBox.scrollHeight + "px";
 
   sizeToContent();
 
   if (gReplacedWindow && !gReplacedWindow.closed) {
     moveWindowToReplace(gReplacedWindow);