From 1c7af0de6c72b2557783489254f8101cc1184f61 Mon Sep 17 00:00:00 2001
From: Doug Bunting <dougbu@microsoft.com>
Date: Fri, 23 Oct 2015 14:48:13 -0700
Subject: [PATCH] Improve `[HtmlTargetElement]` doc comments - #562

---
 .../TagHelpers/HtmlTargetElementAttribute.cs  | 30 ++++++++-----------
 .../TagHelpers/TagHelperOutput.cs             |  1 -
 2 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/HtmlTargetElementAttribute.cs b/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/HtmlTargetElementAttribute.cs
index 655951cf6c..7a3f789e4f 100644
--- a/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/HtmlTargetElementAttribute.cs
+++ b/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/HtmlTargetElementAttribute.cs
@@ -15,47 +15,43 @@ namespace Microsoft.AspNet.Razor.TagHelpers
         public const string ElementCatchAllTarget = TagHelperDescriptorProvider.ElementCatchAllTarget;
 
         /// <summary>
-        /// Instantiates a new instance of the <see cref="HtmlTargetElementAttribute"/> class with <see cref="Tag"/>
-        /// set to <c>*</c>.
+        /// Instantiates a new instance of the <see cref="HtmlTargetElementAttribute"/> class that targets all HTML
+        /// elements with the required <see cref="Attributes"/>.
         /// </summary>
-        /// <remarks>A <c>*</c> <see cref="Tag"/> value indicates an <see cref="ITagHelper"/>
-        /// that targets all HTML elements with the required <see cref="Attributes"/>.</remarks>
+        /// <remarks><see cref="Tag"/> is set to <c>*</c>.</remarks>
         public HtmlTargetElementAttribute()
             : this(ElementCatchAllTarget)
         {
         }
 
         /// <summary>
-        /// Instantiates a new instance of the <see cref="HtmlTargetElementAttribute"/> class.
+        /// Instantiates a new instance of the <see cref="HtmlTargetElementAttribute"/> class with the given
+        /// <paramref name="tag"/> as its <see cref="Tag"/> value.
         /// </summary>
         /// <param name="tag">
         /// The HTML tag the <see cref="ITagHelper"/> targets.
         /// </param>
-        /// <remarks>A <c>*</c> <paramref name="tag"/> value indicates an <see cref="ITagHelper"/>
-        /// that targets all HTML elements with the required <see cref="Attributes"/>.</remarks>
+        /// <remarks>A <c>*</c> <paramref name="tag"/> value indicates this <see cref="ITagHelper"/>
+        /// targets all HTML elements with the required <see cref="Attributes"/>.</remarks>
         public HtmlTargetElementAttribute(string tag)
         {
             Tag = tag;
         }
 
         /// <summary>
-        /// The HTML tag the <see cref="ITagHelper"/> targets.
+        /// The HTML tag the <see cref="ITagHelper"/> targets. A <c>*</c> value indicates this <see cref="ITagHelper"/>
+        /// targets all HTML elements with the required <see cref="Attributes"/>.
         /// </summary>
-        /// <remarks>A <c>*</c> <see cref="Tag"/> value indicates an <see cref="ITagHelper"/>
-        /// that targets all HTML elements with the required <see cref="Attributes"/>.</remarks>
         public string Tag { get; }
 
         /// <summary>
         /// A comma-separated <see cref="string"/> of attribute names the HTML element must contain for the
-        /// <see cref="ITagHelper"/> to run.
+        /// <see cref="ITagHelper"/> to run. <c>*</c> at the end of an attribute name acts as a prefix match.
         /// </summary>
-        /// <remarks>
-        /// <c>*</c> at the end of an attribute name acts as a prefix match.
-        /// </remarks>
         public string Attributes { get; set; }
 
         /// <summary>
-        /// The expected tag structure.
+        /// The expected tag structure. Defaults to <see cref="TagStructure.Unspecified"/>.
         /// </summary>
         /// <remarks>
         /// If <see cref="TagStructure.Unspecified"/> and no other tag helpers applying to the same element specify
@@ -81,9 +77,9 @@ namespace Microsoft.AspNet.Razor.TagHelpers
         public TagStructure TagStructure { get; set; }
 
         /// <summary>
-        /// The required HTML element name of the direct parent.
+        /// The required HTML element name of the direct parent. A <c>null</c> value indicates any HTML element name is
+        /// allowed.
         /// </summary>
-        /// <remarks>A <c>null</c> value indicates any HTML element name is appropriate.</remarks>
         public string ParentTag { get; set; }
     }
 }
\ No newline at end of file
diff --git a/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/TagHelperOutput.cs b/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/TagHelperOutput.cs
index b7816fab02..af8eca767c 100644
--- a/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/TagHelperOutput.cs
+++ b/src/Microsoft.AspNet.Razor.Runtime/TagHelpers/TagHelperOutput.cs
@@ -3,7 +3,6 @@
 
 using System;
 using System.Threading.Tasks;
-using Microsoft.AspNet.Razor.Runtime.TagHelpers;
 
 namespace Microsoft.AspNet.Razor.TagHelpers
 {