Merge pull request #2279 from peppy/fix-settings-notification-verlay

Only allow settings or notifications to be visible at once
ppy · Mar 22, 2018 · a6bcb0b · a6bcb0b
2 parents 61ef635 + e5f8353
commit a6bcb0b
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/osu.Game/OsuGame.cs b/osu.Game/OsuGame.cs
@@ -302,6 +302,21 @@ protected override void LoadComplete()
                 };
             }
 
+            var singleDisplaySideOverlays = new OverlayContainer[] { settings, notifications };
+            foreach (var overlay in singleDisplaySideOverlays)
+            {
+                overlay.StateChanged += state =>
+                {
+                    if (state == Visibility.Hidden) return;
+
+                    foreach (var c in singleDisplaySideOverlays)
+                    {
+                        if (c == overlay) continue;
+                        c.State = Visibility.Hidden;
+                    }
+                };
+            }
+
             // eventually informational overlays should be displayed in a stack, but for now let's only allow one to stay open at a time.
             var informationalOverlays = new OverlayContainer[] { beatmapSetOverlay, userProfile };
             foreach (var overlay in informationalOverlays)