Skip to content

fix(pptx): honor noFill text outlines in HTML preview - #349

Open
y4ho0 wants to merge 1 commit into
iOfficeAI:mainfrom
y4ho0:fix/pptx-nofill-text-outline
Open

fix(pptx): honor noFill text outlines in HTML preview#349
y4ho0 wants to merge 1 commit into
iOfficeAI:mainfrom
y4ho0:fix/pptx-nofill-text-outline

Conversation

@y4ho0

@y4ho0 y4ho0 commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • skip CSS glyph strokes when a run-level text outline explicitly contains a:noFill
  • preserve the existing rendering path for solid and implicit text outlines

Scope

This addresses only the a:rPr/a:ln/a:noFill text-outline item in #341. The connector, inherited alignment, and equation-rendering items in that issue remain out of scope.

Refs #341

Validation

  • dotnet build src/officecli/officecli.csproj --configuration Release --no-restore — passed with 0 errors (one pre-existing nullable warning in ExcelHandler.SheetShift.cs)
  • rendered the public sample.pptx from [PPTX] view html does not preserve several drawing, text, and equation properties #341: before this change the affected run emitted a black -webkit-text-stroke; after the change it emits no text stroke
  • negative control: replacing the same outline with a 1 pt red solid outline still emits -webkit-text-stroke:1.33px #FF0000
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant