[#173] Make error messages more noticeable.

Make "error" font red and increase message time on screen.
Igalia · Mar 6, 2023 · 148f3d7 · 148f3d7
1 parent be96c54
commit 148f3d7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/web/include/App.js b/web/include/App.js
@@ -145,17 +145,9 @@ Ext.extend(Ext.App, Ext.util.Observable, {
      * @param {Bool} status
      */
     addMessage : function(status, msg) {
-        var delay = 3;    // <-- default delay of msg box is 1 second.
+        var delay = 3;    // <-- default delay
         if (status == false) {
-            delay = 5;    // <-- when status is error, msg box delay is 3 seconds.
-        }
-        // add some smarts to msg's duration (div by 13.3 between 3 & 9 seconds)
-        delay = msg.length / 13.3;
-        if (delay < 3) {
-            delay = 3;
-        }
-        else if (delay > 9) {
-            delay = 9;
+            delay = 9;    // <-- when status is error, msg box delay is increased
         }
 
         this.msgCt.alignTo(document, 't-t');

diff --git a/web/include/phpreport.css b/web/include/phpreport.css
@@ -130,6 +130,11 @@ div#tasks textarea {
     margin-bottom: 10px;
 }
 
+/* ExtJS message box */
+.icon-status-error {
+    color: #cc0f35;
+}
+
 .message.error {
     background-color: #feecf0;
     border: 1px solid #cc0f35;