Merge remote-tracking branch 'origin/make-notifications-quieter' into release

osu.Desktop/Overlays/VersionManager.cs

@@ -111,6 +111,8 @@ namespace osu.Desktop.Overlays
 
         private class UpdateCompleteNotification : SimpleNotification
         {
+            public override bool IsImportant => true;
+
             public UpdateCompleteNotification(string version, Action<string> openUrl = null)
             {
                 Text = $"You are now running osu!lazer {version}.\nClick to see what's new!";

osu.Game/Database/ArchiveModelManager.cs

@@ -116,7 +116,7 @@ namespace osu.Game.Database
         /// <param name="paths">One or more archive locations on disk.</param>
         public void Import(params string[] paths)
         {
-            var notification = new ProgressNotification
+            var notification = new ImportNotification
             {
                 Text = "Import is initialising...",
                 Progress = 0,
@@ -407,5 +407,10 @@ namespace osu.Game.Database
                 return new LegacyFilesystemReader(path);
             throw new InvalidFormatException($"{path} is not a valid archive");
         }
+
+        private class ImportNotification : ProgressNotification
+        {
+            public override bool IsImportant => true;
+        }
     }
 }

osu.Game/Overlays/NotificationOverlay.cs

@@ -128,7 +128,8 @@ namespace osu.Game.Overlays
             var section = sections.Children.FirstOrDefault(s => s.AcceptTypes.Any(accept => accept.IsAssignableFrom(ourType)));
             section?.Add(notification, notification.DisplayOnTop ? -runningDepth : runningDepth);
 
-            State = Visibility.Visible;
+            if (notification.IsImportant)
+                State = Visibility.Visible;
 
             updateCounts();
         });

osu.Game/Overlays/Notifications/Notification.cs

@@ -23,6 +23,11 @@ namespace osu.Game.Overlays.Notifications
         /// </summary>
         public event Action Closed;
 
+        /// <summary>
+        /// Whether this notification should forcefully display itself.
+        /// </summary>
+        public virtual bool IsImportant => false;
+
         /// <summary>
         /// Run on user activating the notification. Return true to close.
         /// </summary>