From d4f5d0c8787088c31753fc054a5e96249c0736ca Mon Sep 17 00:00:00 2001
From: Jamie Taylor <me@nekodex.net>
Date: Fri, 7 Jul 2023 01:06:09 +0900
Subject: [PATCH] Revert "Remove sound from notification closing/hiding"

This reverts commit 244f3c6098bb27b66f5ff7fb8c76f38f56cfb4cd.
---
 osu.Game/Overlays/NotificationOverlay.cs | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/osu.Game/Overlays/NotificationOverlay.cs b/osu.Game/Overlays/NotificationOverlay.cs
index ceb5bdd1e9..21027b0931 100644
--- a/osu.Game/Overlays/NotificationOverlay.cs
+++ b/osu.Game/Overlays/NotificationOverlay.cs
@@ -229,7 +229,14 @@ namespace osu.Game.Overlays
             mainContent.FadeEdgeEffectTo(0, WaveContainer.DISAPPEAR_DURATION, Easing.In);
         }
 
-        private void notificationClosed() => Schedule(updateCounts);
+        private void notificationClosed() => Schedule(() =>
+        {
+            updateCounts();
+
+            // this debounce is currently shared between popin/popout sounds, which means one could potentially not play when the user is expecting it.
+            // popout is constant across all notification types, and should therefore be handled using playback concurrency instead, but seems broken at the moment.
+            playDebouncedSample("UI/overlay-pop-out");
+        });
 
         private void playDebouncedSample(string sampleName)
         {