-removed unused variables and summaries

-make new Show method non-generic

Author: corvinsz

src/MaterialDesignThemes.Wpf/DialogHost.cs

@@ -94,8 +94,6 @@ public class DialogHost : ContentControl
     private DialogClosedEventHandler? _attachedDialogClosedEventHandler;
     private IInputElement? _restoreFocusDialogClose;
     private Action? _currentSnackbarMessageQueueUnPauseAction;
-    private Thickness _previousDialogMargin;
-    private double _previousDialogContentUniformCornerRadius;
 
     static DialogHost()
     {
@@ -108,7 +106,6 @@ static DialogHost()
     /// Shows a modal dialog. To use, a <see cref="DialogHost"/> instance must be in a visual tree (typically this may be specified towards the root of a Window's XAML).
     /// </summary>
     /// <param name="content">Content to show (can be a control or view model).</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content)
         => Show(content, null, null);
@@ -118,7 +115,6 @@ static DialogHost()
     /// </summary>
     /// <param name="content">Content to show (can be a control or view model).</param>        
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, DialogOpenedEventHandler? openedEventHandler)
         => Show(content, null, openedEventHandler, null);
@@ -128,7 +124,6 @@ static DialogHost()
     /// </summary>
     /// <param name="content">Content to show (can be a control or view model).</param>
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, DialogClosingEventHandler? closingEventHandler)
         => Show(content, null, null, closingEventHandler);
@@ -139,7 +134,6 @@ static DialogHost()
     /// <param name="content">Content to show (can be a control or view model).</param>        
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, DialogOpenedEventHandler? openedEventHandler, DialogClosingEventHandler? closingEventHandler)
         => Show(content, null, openedEventHandler, closingEventHandler);
@@ -151,7 +145,6 @@ static DialogHost()
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closedEventHandler">Allows access to closed event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, DialogOpenedEventHandler? openedEventHandler, DialogClosingEventHandler? closingEventHandler, DialogClosedEventHandler? closedEventHandler)
         => Show(content, null, openedEventHandler, closingEventHandler, closedEventHandler);
@@ -161,9 +154,8 @@ static DialogHost()
     /// </summary>
     /// <param name="content">Content to show (can be a control or view model).</param>
     /// <param name="dialogIdentifier"><see cref="Identifier"/> of the instance where the dialog should be shown. Typically this will match an identifier set in XAML. <c>null</c> is allowed.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
-    public static Task<object?> Show(object content, object dialogIdentifier, bool isFullscreenDialog = false)
+    public static Task<object?> Show(object content, object dialogIdentifier)
         => Show(content, dialogIdentifier, null, null);
 
     /// <summary>
@@ -172,7 +164,6 @@ static DialogHost()
     /// <param name="content">Content to show (can be a control or view model).</param>
     /// <param name="dialogIdentifier"><see cref="Identifier"/> of the instance where the dialog should be shown. Typically this will match an identifier set in XAML. <c>null</c> is allowed.</param>
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, object dialogIdentifier, DialogOpenedEventHandler? openedEventHandler)
         => Show(content, dialogIdentifier, openedEventHandler, null);
@@ -183,7 +174,6 @@ static DialogHost()
     /// <param name="content">Content to show (can be a control or view model).</param>
     /// <param name="dialogIdentifier"><see cref="Identifier"/> of the instance where the dialog should be shown. Typically this will match an identifier set in XAML. <c>null</c> is allowed.</param>        
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, object dialogIdentifier, DialogClosingEventHandler? closingEventHandler)
         => Show(content, dialogIdentifier, null, closingEventHandler);
@@ -195,7 +185,6 @@ static DialogHost()
     /// <param name="dialogIdentifier"><see cref="Identifier"/> of the instance where the dialog should be shown. Typically this will match an identifier set in XAML. <c>null</c> is allowed.</param>
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static Task<object?> Show(object content, object? dialogIdentifier, DialogOpenedEventHandler? openedEventHandler, DialogClosingEventHandler? closingEventHandler)
         => Show(content, dialogIdentifier, openedEventHandler, closingEventHandler, null);
@@ -208,7 +197,6 @@ static DialogHost()
     /// <param name="openedEventHandler">Allows access to opened event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closingEventHandler">Allows access to closing event which would otherwise have been subscribed to on a instance.</param>
     /// <param name="closedEventHandler">Allows access to closed event which would otherwise have been subscribed to on a instance.</param>
-    /// <param name="isFullscreenDialog">Controls whether the dialog is fullscreen or not.</param>
     /// <returns>Task result is the parameter used to close the dialog, typically what is passed to the <see cref="CloseDialogCommand"/> command.</returns>
     public static async Task<object?> Show(object content, object? dialogIdentifier, DialogOpenedEventHandler? openedEventHandler, DialogClosingEventHandler? closingEventHandler, DialogClosedEventHandler? closedEventHandler)
     {
@@ -326,17 +314,17 @@ private static DialogHost GetInstance(object? dialogIdentifier)
         return result;
     }
 
-    public static Task<T?> Show<T>(object? content, DialogOptions options)
-        => Show<T>(content, null, options);
+    public static Task<object?> Show(object? content, DialogOptions options)
+        => Show(content, null, options);
 
-    public static async Task<T?> Show<T>(object? content, object? dialogIdentifier, DialogOptions options)
+    public static async Task<object?> Show(object? content, object? dialogIdentifier, DialogOptions options)
     {
 #if NET7_0_OR_GREATER
         ArgumentNullException.ThrowIfNull(content);
 #else
         if (content is null) throw new ArgumentNullException(nameof(content));
 #endif
-        return await GetInstance(dialogIdentifier).ShowInternal<T>(content, options);
+        return await GetInstance(dialogIdentifier).ShowInternal(content, options);
     }
 
     private void ApplyDialogOptions(DialogOptions options)
@@ -413,7 +401,7 @@ private void RevertDialogOptions(DialogOptions options)
         }
     }
 
-    internal async Task<T?> ShowInternal<T>(object content, DialogOptions options)
+    internal async Task<object?> ShowInternal(object content, DialogOptions options)
     {
         if (IsOpen)
             throw new InvalidOperationException("DialogHost is already open.");
@@ -440,7 +428,7 @@ private void RevertDialogOptions(DialogOptions options)
 
         RevertDialogOptions(options);
 
-        return result is null ? default : (T)result;
+        return result;
     }
 
     private void OnCloseOnClickAway(object sender, MouseButtonEventArgs e)