Fix important notifications interrupting first-run setup

This commit is contained in:
Salman Ahmed 2022-06-15 03:36:38 +03:00
parent 1171d44ad9
commit ddeee09a51

View File

@ -35,6 +35,9 @@ namespace osu.Game.Overlays
[Resolved] [Resolved]
private AudioManager audio { get; set; } private AudioManager audio { get; set; }
[Resolved(canBeNull: true)]
private FirstRunSetupOverlay firstRunSetup { get; set; }
[BackgroundDependencyLoader] [BackgroundDependencyLoader]
private void load() private void load()
{ {
@ -130,7 +133,9 @@ namespace osu.Game.Overlays
var section = sections.Children.FirstOrDefault(s => s.AcceptTypes.Any(accept => accept.IsAssignableFrom(ourType))); var section = sections.Children.FirstOrDefault(s => s.AcceptTypes.Any(accept => accept.IsAssignableFrom(ourType)));
section?.Add(notification, notification.DisplayOnTop ? -runningDepth : runningDepth); section?.Add(notification, notification.DisplayOnTop ? -runningDepth : runningDepth);
if (notification.IsImportant) // we don't want important notifications interrupting user on first-run setup.
// (this can happen when importing beatmaps inside setup, which posts import notifications)
if (notification.IsImportant && firstRunSetup?.State.Value != Visibility.Visible)
Show(); Show();
updateCounts(); updateCounts();