From 339c17ba1dd4cad3bfb0a5a9f96742f5789675c8 Mon Sep 17 00:00:00 2001
From: Loni Tra <lonitra@microsoft.com>
Date: Wed, 2 Oct 2024 16:17:14 -0700
Subject: [PATCH] Apply suggestions from code review
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Carlos Sánchez López <1175054+carlossanlop@users.noreply.github.com>
---
 xml/System.Drawing.Imaging.Effects/Effect.xml | 5 +++--
 xml/System.Drawing/SystemColors.xml           | 6 +++---
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/xml/System.Drawing.Imaging.Effects/Effect.xml b/xml/System.Drawing.Imaging.Effects/Effect.xml
index df03e34af64..663e141612b 100644
--- a/xml/System.Drawing.Imaging.Effects/Effect.xml
+++ b/xml/System.Drawing.Imaging.Effects/Effect.xml
@@ -65,8 +65,9 @@
         <Parameter Name="disposing" Type="System.Boolean" />
       </Parameters>
       <Docs>
-        <param name="disposing"><see langword="true"/> if managed resources should be disposed; otherwise, <see langword="false"/>.</param>
-        <summary>Releases the resources that are used by the object.</summary>
+        <param name="disposing">
+          <see langword="true" /> to release both managed and unmanaged resources; false to release only unmanaged resources.</param>
+        <summary>Releases the unmanaged resources used by the <see cref="T:System.Drawing.Imaging.Effects.Effect" /> and optionally releases the managed resources.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
diff --git a/xml/System.Drawing/SystemColors.xml b/xml/System.Drawing/SystemColors.xml
index 6c21d9197b4..434381a922e 100644
--- a/xml/System.Drawing/SystemColors.xml
+++ b/xml/System.Drawing/SystemColors.xml
@@ -1345,10 +1345,10 @@
       </ReturnValue>
       <Docs>
         <summary>Gets or sets whether the alternative color set is used.</summary>
-        <value>When <see langword="true"/>, system <see cref="KnownColor"/> values will return the alternative color set (as returned by <see cref="SystemColors"/> statics or <see cref="Color.FromKnownColor(KnownColor)"/>). This is currently "dark mode" variants of the system colors.</value>
+        <value>When <see langword="true"/>, system <see cref="T:System.Drawing.KnownColor"/> values will return the alternative color set (as returned by <see cref="T:System.Drawing.SystemColors"/> statics or <see cref="M:System.Drawing.Color.FromKnownColor(System.Drawing.KnownColor)"/>). This is currently "dark mode" variants of the system colors.</value>
         <remarks>
-          <para><see cref="KnownColor"/> <see cref="Color"/> values are always looked up every time you use them and do not retain any other context. As such, existing <see cref="Color"/> values will change when this property is set.</para>
-          <para>On Windows, system <see cref="KnownColor"/> values will always return the current Windows color when the OS has a high contrast theme enabled.</para>
+          <para><see cref="T:System.Drawing.KnownColor"/> <see cref="T:System.Drawing.Color"/> values are always looked up every time you use them and do not retain any other context. As such, existing <see cref="T:System.Drawing.Color"/> values will change when this property is set.</para>
+          <para>On Windows, system <see cref="T:System.Drawing.KnownColor"/> values will always return the current Windows color when the OS has a high contrast theme enabled.</para>
         </remarks>
       </Docs>
     </Member>