Merge pull request #374 from X-Sharp/master

Updated XML comment for MessageBox.ShowConfirm() and suppressed a war…
VsixCommunity · Sep 14, 2022 · e81bde6 · e81bde6
2 parents 90379cf + 3fb426b
commit e81bde6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/toolkit/Community.VisualStudio.Toolkit.Shared/Notifications/InfoBar.cs b/src/toolkit/Community.VisualStudio.Toolkit.Shared/Notifications/InfoBar.cs
@@ -172,6 +172,7 @@ public void Close()
         void IVsInfoBarUIEvents.OnClosed(IVsInfoBarUIElement infoBarUIElement)
         {
             IsVisible = false;
+            ThreadHelper.ThrowIfNotOnUIThread();
             _uiElement?.Unadvise(_listenerCookie);
         }
 

diff --git a/src/toolkit/Community.VisualStudio.Toolkit.Shared/Notifications/MessageBox.cs b/src/toolkit/Community.VisualStudio.Toolkit.Shared/Notifications/MessageBox.cs
@@ -95,7 +95,7 @@ public async Task<MessageBoxResult> ShowWarningAsync(string line1, string line2
         }
 
         /// <summary>
-        /// Shows a yes/no/cancel message box.
+        /// Shows a yes/no message box.
         /// </summary>
         /// <returns>true if the user clicks the 'Yes' button.</returns>
         public bool ShowConfirm(string line1, string line2 = "")
@@ -105,7 +105,7 @@ public bool ShowConfirm(string line1, string line2 = "")
         }
 
         /// <summary>
-        /// Shows a yes/no/cancel message box.
+        /// Shows a yes/no message box.
         /// </summary>
         /// <returns>true if the user clicks the 'Yes' button.</returns>
         public async Task<bool> ShowConfirmAsync(string line1, string line2 = "")