From 2617db3f5233624de4b744b84bd7ddab7418f49e Mon Sep 17 00:00:00 2001 From: BrensNina <135912553+BrensNina@users.noreply.github.com> Date: Tue, 30 Dec 2025 13:31:01 -0500 Subject: [PATCH] Clarify DialogHost.Close parameter documentation The documentation for DialogHost.Close(...) described the `parameter` argument as being "to provide to close handler", which is misleading and incomplete. Internally, DialogHost.Close forwards the value directly to DialogSession.Close(parameter). In DialogSession.Close, the parameter is correctly documented as the dialog result value, which is returned by DialogHost.Show(...) and exposed through DialogClosingEventArgs.Parameter. This change aligns the DialogHost.Close documentation with the existing DialogSession.Close documentation, making it clear that the parameter represents the dialog result and is not solely intended for close handlers. --- src/MaterialDesignThemes.Wpf/DialogHost.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/MaterialDesignThemes.Wpf/DialogHost.cs b/src/MaterialDesignThemes.Wpf/DialogHost.cs index 1fb5688e3a..4e919f2552 100644 --- a/src/MaterialDesignThemes.Wpf/DialogHost.cs +++ b/src/MaterialDesignThemes.Wpf/DialogHost.cs @@ -186,7 +186,7 @@ public static void Close(object? dialogIdentifier) /// Close a modal dialog. /// /// of the instance where the dialog should be closed. Typically this will match an identifier set in XAML. - /// to provide to close handler + /// Value returned by DialogHost.ShowDialog(...) or passed to close handler if one is provided. public static void Close(object? dialogIdentifier, object? parameter) { DialogHost dialogHost = GetInstance(dialogIdentifier);