Compare commits

..

2 Commits

Author SHA1 Message Date
487261592e ava: Fix regression on title updater and dlc manager window caused by precious commit 2023-07-21 22:50:10 +02:00
9e04e6cba1 Ava UI: Remove IsActive checks from dialog methods (#5456)
* Remove `IsActive` checks from dialog methods

* Remove old windows bandaid

* Remove null dialog code path entirely and return nothing.
2023-07-21 12:24:13 +01:00
2 changed files with 7 additions and 5 deletions

View File

@ -53,8 +53,6 @@ namespace Ryujinx.Ava.UI.Applet
bool opened = false;
_parent.Activate();
UserResult response = await ContentDialogHelper.ShowDeferredContentDialog(_parent,
title,
message,

View File

@ -315,8 +315,10 @@ namespace Ryujinx.Ava.UI.Helpers
Window parent = GetMainWindow();
if (parent is { IsActive: true } and MainWindow window && window.ViewModel.IsGameRunning)
if (parent is MainWindow window)
{
parent.Activate();
contentDialogOverlayWindow = new()
{
Height = parent.Bounds.Height,
@ -369,7 +371,9 @@ namespace Ryujinx.Ava.UI.Helpers
}
else
{
result = await contentDialog.ShowAsync();
result = ContentDialogResult.None;
Logger.Warning?.Print(LogClass.Ui, "Content dialog overlay failed to populate. Default value has been returned.");
}
return result;
@ -390,7 +394,7 @@ namespace Ryujinx.Ava.UI.Helpers
{
foreach (Window item in al.Windows)
{
if (item.IsActive && item is MainWindow window)
if (item is MainWindow window)
{
return window;
}