New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make beatmap overlay download buttons work #1506
Conversation
naoey
commented
Nov 15, 2017
- Add noVideo option to DownloadBeatmapSetRequest
- Make Download fire an event with new download instead of returning it
- Add noVideo option to DownloadBeatmapSetRequest - Make Download fire an event with new download instead of returning it
@@ -109,6 +103,8 @@ private void load(APIAccess api, BeatmapManager beatmaps, OsuColour colours, Not | |||
|
|||
if (downloadRequest != null) | |||
attachDownload(downloadRequest); | |||
|
|||
beatmaps.BeatmapDownloadBegan += attachDownload; |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
{ | ||
this.api = api; | ||
this.rulesets = rulesets; | ||
|
||
manager.BeatmapSetAdded += beatmap => |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
- Also unbind event handlers - Remove unused field
d8aaf81
to
97c5956
Compare
private void handleBeatmapAdd(BeatmapSetInfo beatmap) | ||
{ | ||
if (beatmap.OnlineBeatmapSetID == BeatmapSet.OnlineBeatmapSetID) | ||
downloadButtonsContainer.FadeOut(transition_duration); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Going to merge this even though the full behaviour I mentioned in my comments isn't yet implemented. @naoey pointed out on discord that he did try to add this and didn't have much luck, so let's leave that for a future PR. |