Fix override

Author: theletterf

docs/syntax/images.md

@@ -68,14 +68,15 @@ Titles are optional making this the minimal syntax required:
 ![Elasticsearch](/syntax/images/observability.png)
 ```
 
-When no title is explicitly provided, the alt text is automatically used as the title.
-
-If you want a different title, you can supply it as an optional argument:
+For inline images, the alt text always overrides any title specified in the Markdown. This ensures consistent accessibility where both the `alt` and `title` attributes contain the same descriptive text.
 
 ```markdown
 ![Elasticsearch](/syntax/images/observability.png "Different title")
 ```
 
+![Elasticsearch](/syntax/images/observability.png "Different title")
+
+
 ### Inline image sizing
 
 Image sizing is specified through the title argument. You can specify just the size without needing to provide a redundant title:
@@ -85,7 +86,7 @@ Image sizing is specified through the title argument. You can specify just the s
 ![alt](img.png "=W")
 ```
 
-In this case, the alt text will automatically be used as the title, and the size parameters will be applied.
+In this case, the alt text will be used as both the `alt` and `title` attributes, and the size parameters will be applied.
 
 `W` and `H` can be either an absolute number in pixels or a number followed by `%` to indicate relative sizing.
 

src/Elastic.Markdown/Myst/Renderers/HtmxLinkInlineRenderer.cs

@@ -20,7 +20,6 @@ protected override void Write(HtmlRenderer renderer, LinkInline link)
 	{
 		if (renderer.EnableHtmlForInline && !link.IsImage)
 		{
-			// ReSharper disable once UnusedVariable
 			if (link.GetData(nameof(ParserContext.CurrentUrlPath)) is not string currentUrl)
 			{
 				base.Write(renderer, link);
@@ -70,9 +69,42 @@ protected override void Write(HtmlRenderer renderer, LinkInline link)
 
 			_ = renderer.Write("</a>");
 		}
+		else if (link.IsImage)
+		{
+			// Handle inline images with ALT override logic
+			WriteImage(renderer, link);
+		}
 		else
 			base.Write(renderer, link);
 	}
+
+	private static void WriteImage(HtmlRenderer renderer, LinkInline link)
+	{
+		_ = renderer.Write("<img src=\"");
+		_ = renderer.WriteEscapeUrl(link.GetDynamicUrl?.Invoke() ?? link.Url);
+		_ = renderer.Write('"');
+
+		// Write alt text using WriteChildren to ensure substitutions are processed
+		if (link.FirstChild != null)
+		{
+			_ = renderer.Write(" alt=\"");
+			renderer.WriteChildren(link);
+			_ = renderer.Write('"');
+		}
+
+		// Write any additional attributes (like width/height from styling instructions)
+		_ = renderer.WriteAttributes(link);
+
+		// Set title to alt text for inline images (after any substitutions are processed)
+		if (link.FirstChild != null)
+		{
+			_ = renderer.Write(" title=\"");
+			renderer.WriteChildren(link);
+			_ = renderer.Write('"');
+		}
+
+		_ = renderer.Write(" />");
+	}
 }
 
 public static class CustomLinkInlineRendererExtensions

tests/Elastic.Markdown.Tests/Inline/SubstitutionTest.cs

@@ -203,7 +203,7 @@ public class ReplaceInImageTitle(ITestOutputHelper output) : InlineTest(output,
 	[Fact]
 	public void OnlySeesGlobalVariable() =>
 		Html.Should().NotContain("title=\"{{hello-world}}\"")
-			.And.Contain("title=\"Hello World\"");
+			.And.Contain("title=\"Observability\"");
 }
 
 public class MutationOperatorTest(ITestOutputHelper output) : InlineTest(output,

tests/authoring/Inline/InlineAppliesTo.fs

@@ -15,7 +15,7 @@ type ``static path to image`` () =
     [<Fact>]
     let ``validate HTML: generates link and alt attr`` () =
         markdown |> convertsToHtml """
-            <p><img src="/_static/img/observability.png" alt="Elasticsearch" /></p>
+            <p><img src="/_static/img/observability.png" alt="Elasticsearch" title="Elasticsearch" /></p>
         """
 
 type ``relative path to image`` () =
@@ -26,7 +26,7 @@ type ``relative path to image`` () =
     [<Fact>]
     let ``validate HTML: preserves relative path`` () =
         markdown |> convertsToHtml """
-            <p><img src="/_static/img/observability.png" alt="Elasticsearch" /></p>
+            <p><img src="/_static/img/observability.png" alt="Elasticsearch" title="Elasticsearch" /></p>
         """
 
 type ``supplying a tittle`` () =
@@ -37,7 +37,7 @@ type ``supplying a tittle`` () =
     [<Fact>]
     let ``validate HTML: includes title`` () =
         markdown |> convertsToHtml """
-            <p><img src="/_static/img/observability.png" alt="Elasticsearch" title="Hello world" /></p>
+            <p><img src="/_static/img/observability.png" alt="Elasticsearch" title="Elasticsearch" /></p>
         """
 
 type ``supplying a tittle with width and height`` () =
@@ -48,7 +48,7 @@ type ``supplying a tittle with width and height`` () =
     [<Fact>]
     let ``validate HTML: does not include width and height in title`` () =
         markdown |> convertsToHtml """
-            <p><img src="/obs.png" width="250px" height="400px" alt="o" title="Title"/></p>
+            <p><img src="/obs.png" width="250px" height="400px" alt="o" title="o"/></p>
         """
 
 type ``supplying a tittle with width and height in percentage`` () =
@@ -59,7 +59,7 @@ type ``supplying a tittle with width and height in percentage`` () =
     [<Fact>]
     let ``validate HTML: does not include width and height in title`` () =
         markdown |> convertsToHtml """
-            <p><img src="/obs.png" width="50%" height="40%" alt="o" title="Title"/></p>
+            <p><img src="/obs.png" width="50%" height="40%" alt="o" title="o"/></p>
         """
 type ``supplying a tittle with width only`` () =
     static let markdown = Setup.Markdown """
@@ -69,5 +69,5 @@ type ``supplying a tittle with width only`` () =
     [<Fact>]
     let ``validate HTML: sets height to width if not supplied`` () =
         markdown |> convertsToHtml """
-            <p><img src="/obs.png" width="30%" height="30%" alt="o" title="Title"/></p>
+            <p><img src="/obs.png" width="30%" height="30%" alt="o" title="o"/></p>
         """