From 77dbf3e4991dec35cecb7e8813e30e2f49dbfd78 Mon Sep 17 00:00:00 2001
From: Yahoo <119646314+y4ho0@users.noreply.github.com>
Date: Sat, 29 Aug 2026 17:34:43 +0800
Subject: [PATCH] fix(pptx): honor noFill text outlines in HTML preview

---
 .../Handlers/Pptx/PowerPointHandler.HtmlPreview.Text.cs  | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/officecli/Handlers/Pptx/PowerPointHandler.HtmlPreview.Text.cs b/src/officecli/Handlers/Pptx/PowerPointHandler.HtmlPreview.Text.cs
index f8c6743ca..5fa391355 100644
--- a/src/officecli/Handlers/Pptx/PowerPointHandler.HtmlPreview.Text.cs
+++ b/src/officecli/Handlers/Pptx/PowerPointHandler.HtmlPreview.Text.cs
@@ -1133,11 +1133,12 @@ private static void RenderRun(StringBuilder sb, Drawing.Run run, Dictionary<stri
             // Run-level text outline (a:rPr/a:ln). PowerPoint strokes each glyph
             // edge; Chromium renders this via -webkit-text-stroke. Width is the
             // a:ln @w in EMU (12700 EMU = 1pt); convert to px (1pt = 4/3 px) so a
-            // 3pt outline reads as a ~4px stroke. Color comes from the a:ln's
-            // solidFill child (default black when absent). paint-order:stroke fill
-            // keeps the fill painted on top so the stroke hugs the glyph outside.
+            // 3pt outline reads as a ~4px stroke. An explicit noFill disables
+            // the stroke; otherwise color comes from solidFill (default black
+            // when absent). paint-order:stroke fill keeps the fill painted on
+            // top so the stroke hugs the glyph outside.
             var runOutline = rp.GetFirstChild<Drawing.Outline>();
-            if (runOutline != null)
+            if (runOutline != null && runOutline.GetFirstChild<Drawing.NoFill>() == null)
             {
                 double strokePx = runOutline.Width?.HasValue == true
                     ? Units.EmuToPt(runOutline.Width.Value) * 4.0 / 3.0