Fix up the suggestons to get it to build again

Author: Warren Buckley

Our.Umbraco.TagHelpers/IsActivePageTagHelper.cs

@@ -1,6 +1,8 @@
 using Microsoft.AspNetCore.Razor.TagHelpers;
+using System.Text.Encodings.Web;
 using Umbraco.Cms.Core.Web;
 using Umbraco.Extensions;
+using Microsoft.AspNetCore.Mvc.TagHelpers;
 
 namespace Our.Umbraco.TagHelpers
 {
@@ -12,9 +14,10 @@ namespace Our.Umbraco.TagHelpers
     /// if the value in the href of the <a> compared to the page being rendered
     /// is the current page or part of an ancestor
     /// </summary>
-    [HtmlTargetElement("a", Attributes = "our-is-active-page")]
+    [HtmlTargetElement("a", Attributes = tagHelperAttributeName)]
     public class IsActivePageTagHelper : TagHelper
     {
+        private const string tagHelperAttributeName = "our-active-class";
         private IUmbracoContextAccessor _umbracoContextAccessor;
 
         public IsActivePageTagHelper(IUmbracoContextAccessor umbracoContextAccessor)
@@ -26,7 +29,7 @@ public IsActivePageTagHelper(IUmbracoContextAccessor umbracoContextAccessor)
         /// The CSS class name you wish to append to the class attribute
         /// on an <a> tag when the page is active/selected
         /// </summary>
-        [HtmlAttributeName("our-is-active-page")]
+        [HtmlAttributeName(tagHelperAttributeName)]
         public string ActiveClassName { get; set; }
 
         public override void Process(TagHelperContext context, TagHelperOutput output)
@@ -62,7 +65,7 @@ public override void Process(TagHelperContext context, TagHelperOutput output)
             if(nodeOfLink.IsAncestorOrSelf(currentPageRendering))
             {
                 // Is active page
-                output.Attributes.AddClass(ActiveClassName, HtmlEncoder.Default);
+                output.AddClass(ActiveClassName, HtmlEncoder.Default);
             }
         }
     }

README.md

@@ -292,7 +292,7 @@ If you want the edit link to open in a new tab, just add the `target="_blank"` a
 <our-edit-link target="_blank">Edit</our-edit-link>
 ```
 
-## `our-is-active-page`
+## `our-active-class`
 This is a tag helper attribute that can be applied to `<a>` element in the razor template or partial. It will use the value inside the attribute and append it to the class attribute of the `<a>`.
 If the link inside the href attribute can be found by its route as a content node, if so then it will check with the current page being rendered if its the same node or an ancestor.
 
@@ -302,7 +302,7 @@ This allows for the navigation item to still have the class added to it when a c
 ```cshtml
 @foreach (var item in Model.Root().Children)
 {
-    <a href="@item.Url()" class="nav-link" our-is-active-page="nav-link--active">@item.Name</a>
+    <a href="@item.Url()" class="nav-link" our-active-class="nav-link--active">@item.Name</a>
 }
 ```