From 5fb044956945915ef28054a04bcc1ceae8a9b9eb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= Date: Mon, 30 Jun 2025 13:47:13 +0200 Subject: [PATCH] Fix error code paths just completely falling apart You can't hide a drawable outside of update thread. --- osu.Game/Overlays/SkinEditor/ExternalEditOverlay.cs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/osu.Game/Overlays/SkinEditor/ExternalEditOverlay.cs b/osu.Game/Overlays/SkinEditor/ExternalEditOverlay.cs index cbf85c6f2b..e4ac157936 100644 --- a/osu.Game/Overlays/SkinEditor/ExternalEditOverlay.cs +++ b/osu.Game/Overlays/SkinEditor/ExternalEditOverlay.cs @@ -115,10 +115,9 @@ namespace osu.Game.Overlays.SkinEditor catch (Exception ex) { Logger.Log($"Failed to initialize external edit operation: {ex}", LoggingTarget.Database, LogLevel.Error); - Schedule(() => showSpinner("Export failed!")); - await Task.Delay(1000).ConfigureAwait(true); setGlobalSkinDisabled(false); - Hide(); + Schedule(() => showSpinner("Export failed!")); + Scheduler.AddDelayed(Hide, 1000); return Task.FromException(ex); } @@ -195,8 +194,7 @@ namespace osu.Game.Overlays.SkinEditor { Logger.Log($"Failed to finish external edit operation: {ex}", LoggingTarget.Database, LogLevel.Error); showSpinner("Import failed!"); - await Task.Delay(1000).ConfigureAwait(true); - Hide(); + Scheduler.AddDelayed(Hide, 1000); setGlobalSkinDisabled(false); taskCompletionSource.SetException(ex); taskCompletionSource = null;