Merge branch 'img-helper-alt-fix' of https://github.com/richarth/Our-Umbraco-TagHelpers into richarth-img-helper-alt-fix

Warren Buckley · Warren Buckley · commit b0fa53cc1c6c · 2023-05-05T12:55:30.000+01:00
diff --git a/Our.Umbraco.TagHelpers/ImgTagHelper.cs b/Our.Umbraco.TagHelpers/ImgTagHelper.cs
@@ -180,12 +180,15 @@ public override void Process(TagHelperContext context, TagHelperOutput output)
                     height = (originalHeight / originalWidth) * width;
                 }
 
-                #region Autogenerate alt text
+                #region Autogenerate alt text if unspecfied
                 if (string.IsNullOrWhiteSpace(ImgAlt))
                 {
                     output.Attributes.Add("alt", GetImageAltText(MediaItem));
                 }
-                #endregion
+                else
+                {
+                    output.Attributes.Add("alt", ImgAlt);
+                }
                 #endregion
             }
             else if (!string.IsNullOrEmpty(FileSource))
@@ -196,11 +199,15 @@ public override void Process(TagHelperContext context, TagHelperOutput output)
 
                 imgSrc = AddQueryToUrl(FileSource, "width", width.ToString());
 
-                #region Autogenerate alt text
+                #region Autogenerate alt text if unspecfied
                 if (string.IsNullOrWhiteSpace(ImgAlt))
                 {
                     output.Attributes.Add("alt", GetImageAltText(FileSource));
                 }
+                else
+                {
+                    output.Attributes.Add("alt", ImgAlt);
+                }
                 #endregion
 
                 #region If width & height are not defined then return a basic <img> with just a src, alt & class (if provided)

Original file line number	Diff line number	Diff line change
`@@ -180,12 +180,15 @@ public override void Process(TagHelperContext context, TagHelperOutput output)`
`180`	`180`	`height = (originalHeight / originalWidth) * width;`
`181`	`181`	`}`
`182`	`182`
`183`		`- #region Autogenerate alt text`
	`183`	`+ #region Autogenerate alt text if unspecfied`
`184`	`184`	`if (string.IsNullOrWhiteSpace(ImgAlt))`
`185`	`185`	`{`
`186`	`186`	`output.Attributes.Add("alt", GetImageAltText(MediaItem));`
`187`	`187`	`}`
`188`		`- #endregion`
	`188`	`+ else`
	`189`	`+ {`
	`190`	`+ output.Attributes.Add("alt", ImgAlt);`
	`191`	`+ }`
`189`	`192`	`#endregion`
`190`	`193`	`}`
`191`	`194`	`else if (!string.IsNullOrEmpty(FileSource))`
`@@ -196,11 +199,15 @@ public override void Process(TagHelperContext context, TagHelperOutput output)`
`196`	`199`
`197`	`200`	`imgSrc = AddQueryToUrl(FileSource, "width", width.ToString());`
`198`	`201`
`199`		`- #region Autogenerate alt text`
	`202`	`+ #region Autogenerate alt text if unspecfied`
`200`	`203`	`if (string.IsNullOrWhiteSpace(ImgAlt))`
`201`	`204`	`{`
`202`	`205`	`output.Attributes.Add("alt", GetImageAltText(FileSource));`
`203`	`206`	`}`
	`207`	`+ else`
	`208`	`+ {`
	`209`	`+ output.Attributes.Add("alt", ImgAlt);`
	`210`	`+ }`
`204`	`211`	`#endregion`
`205`	`212`
`206`	`213`	`#region If width & height are not defined then return a basic <img> with just a src, alt & class (if provided)`