[4.0] fix Media Manager hidden alerts (#32868)

* fix * some todos, if the other alert PRs get merged * CS
joomla · Mar 26, 2021 · 755f836 · 755f836
1 parent c97dddd
commit 755f836
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/administrator/components/com_media/resources/scripts/app/Notifications.es6.js b/administrator/components/com_media/resources/scripts/app/Notifications.es6.js
@@ -4,7 +4,7 @@ class Notifications {
   success(message, options) {
     // eslint-disable-next-line no-use-before-define
     notifications.notify(message, {
-      type: 'message',
+      type: 'message', // @todo rename it to success
       dismiss: true,
       ...options,
     });
@@ -15,7 +15,7 @@ class Notifications {
   error(message, options) {
     // eslint-disable-next-line no-use-before-define
     notifications.notify(message, {
-      type: 'error',
+      type: 'error', // @todo rename it to danger
       dismiss: true,
       ...options,
     });
@@ -30,10 +30,17 @@ class Notifications {
   /* Send a notification */
   // eslint-disable-next-line class-methods-use-this
   notify(message, options) {
+    let timer;
+    if (options.type === 'message') {
+      timer = 3000;
+    }
     Joomla.renderMessages(
       {
         [options.type]: [Joomla.JText._(message)],
       },
+      undefined,
+      true,
+      timer,
     );
   }
 }

diff --git a/build/media_source/system/js/core.es6.js b/build/media_source/system/js/core.es6.js
@@ -675,7 +675,7 @@ window.Joomla.Modal = window.Joomla.Modal || {
       messagesBox.setAttribute('dismiss', 'true');
 
       if (timeout && parseInt(timeout, 10) > 0) {
-        messagesBox.setAttribute('autodismiss', timeout);
+        messagesBox.setAttribute('auto-dismiss', timeout);
       }
 
       // Title