Skip to content

fix: restore pptx mixed-script no-wrap titles#173

Merged
developer0hye merged 2 commits intomainfrom
fix/pptx-nowrap-regression
Mar 14, 2026
Merged

fix: restore pptx mixed-script no-wrap titles#173
developer0hye merged 2 commits intomainfrom
fix/pptx-nowrap-regression

Conversation

@developer0hye
Copy link
Copy Markdown
Owner

What changed

  • restore no-wrap behavior for mixed-script PPTX titles after the text-extraction follow-up
  • keep no-wrap state across styled run boundaries so CJK section titles and mixed-font headers stay on one line
  • add regression coverage for mixed-script no-wrap titles and cross-run heading cases

Why

  • main regressed on the KTB PPTX deck: slide 9 TOC and slides 20/30 headers wrapped again after fix: preserve no-wrap text extraction

Validation

  • cargo test --offline -p office2pdf fixed_page_textbox_tests
  • cargo test --offline -p office2pdf --test pptx_fixtures text_content_test_slides -- --nocapture
  • rebuilt the CLI and visually rechecked slides 9, 20, and 30 in the regenerated PDF

Signed-off-by: Yonghye Kwon <developer.0hye@gmail.com>
Signed-off-by: Yonghye Kwon <developer.0hye@gmail.com>
@developer0hye developer0hye merged commit 3d63ef8 into main Mar 14, 2026
13 of 14 checks passed
@developer0hye developer0hye deleted the fix/pptx-nowrap-regression branch March 14, 2026 16:52
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