From d8aaf810bd7c3eef0d193b88cc4e627b3ae3d135 Mon Sep 17 00:00:00 2001
From: naoey <naoey20@gmail.com>
Date: Thu, 16 Nov 2017 15:36:22 +0530
Subject: [PATCH] Make download buttons disappear instead of closing overlay.

- Also unbind event handlers
---
 osu.Game/Overlays/BeatmapSet/Header.cs  | 15 +++++++++++++++
 osu.Game/Overlays/BeatmapSetOverlay.cs  |  8 +-------
 osu.Game/Overlays/Direct/DirectPanel.cs |  6 ++++++
 3 files changed, 22 insertions(+), 7 deletions(-)

diff --git a/osu.Game/Overlays/BeatmapSet/Header.cs b/osu.Game/Overlays/BeatmapSet/Header.cs
index ed72cf0cc93f..27ef6208bebe 100644
--- a/osu.Game/Overlays/BeatmapSet/Header.cs
+++ b/osu.Game/Overlays/BeatmapSet/Header.cs
@@ -51,6 +51,7 @@ public BeatmapSetInfo BeatmapSet
                 title.Text = BeatmapSet.Metadata.Title;
                 artist.Text = BeatmapSet.Metadata.Artist;
 
+                downloadButtonsContainer.FadeIn();
                 noVideoButtons.FadeTo(BeatmapSet.OnlineInfo.HasVideo ? 0 : 1, transition_duration);
                 videoButtons.FadeTo(BeatmapSet.OnlineInfo.HasVideo ? 1 : 0, transition_duration);
 
@@ -224,6 +225,20 @@ private void load(OsuColour colours, BeatmapManager beatmaps)
         {
             tabsBg.Colour = colours.Gray3;
             this.beatmaps = beatmaps;
+
+            beatmaps.BeatmapSetAdded += handleBeatmapAdd;
+        }
+
+        protected override void Dispose(bool isDisposing)
+        {
+            base.Dispose(isDisposing);
+            beatmaps.BeatmapSetAdded -= handleBeatmapAdd;
+        }
+
+        private void handleBeatmapAdd(BeatmapSetInfo beatmap)
+        {
+            if (beatmap.OnlineBeatmapSetID == BeatmapSet.OnlineBeatmapSetID)
+                downloadButtonsContainer.FadeOut(transition_duration);
         }
 
         private void download(bool noVideo)
diff --git a/osu.Game/Overlays/BeatmapSetOverlay.cs b/osu.Game/Overlays/BeatmapSetOverlay.cs
index 7ae2d323b895..37de470232aa 100644
--- a/osu.Game/Overlays/BeatmapSetOverlay.cs
+++ b/osu.Game/Overlays/BeatmapSetOverlay.cs
@@ -87,16 +87,10 @@ public BeatmapSetOverlay()
         }
 
         [BackgroundDependencyLoader]
-        private void load(APIAccess api, RulesetStore rulesets, BeatmapManager manager)
+        private void load(APIAccess api, RulesetStore rulesets)
         {
             this.api = api;
             this.rulesets = rulesets;
-
-            manager.BeatmapSetAdded += beatmap =>
-            {
-                if (beatmap.OnlineBeatmapSetID == currentBeatmap.OnlineBeatmapSetID)
-                    Hide();
-            };
         }
 
         protected override void PopIn()
diff --git a/osu.Game/Overlays/Direct/DirectPanel.cs b/osu.Game/Overlays/Direct/DirectPanel.cs
index d4f4067fcafc..b8a852896200 100644
--- a/osu.Game/Overlays/Direct/DirectPanel.cs
+++ b/osu.Game/Overlays/Direct/DirectPanel.cs
@@ -107,6 +107,12 @@ private void load(BeatmapManager beatmaps, OsuColour colours, BeatmapSetOverlay
             beatmaps.BeatmapDownloadBegan += attachDownload;
         }
 
+        protected override void Dispose(bool isDisposing)
+        {
+            base.Dispose(isDisposing);
+            beatmaps.BeatmapDownloadBegan -= attachDownload;
+        }
+
         protected override void Update()
         {
             base.Update();