Skip to content

Render a whitespace-only placeholder with its stray bullet - #17

Merged
uvrt merged 2 commits into
mainfrom
gaps-ws-bullet
Oct 13, 2026
Merged

uvrt merged 2 commits into
mainfrom
gaps-ws-bullet

Conversation

@uvrt

@uvrt uvrt commented Oct 11, 2026

Copy link
Copy Markdown
Owner

empty_placeholders reports a placeholder that holds only whitespace and warns about the stray bullet PowerPoint draws for it, in edit view and in its export. The render now draws that bullet too, through ooxml-common 0.13.0 (uvrt/ooxml-common#24), so render_svg shows what the check reports.

  • Detection is unchanged and still XML-based: whitespace-only text, including a no-break space, is reported empty. PowerPoint shows no prompt for it and draws one bullet.
  • A placeholder of empty paragraphs is still reported, and still renders nothing, as in PowerPoint.
  • New test: test_a_whitespace_one_is_reported_and_drawn_with_its_stray_bullet covers three spaces, a tab, an NBSP, and empty paragraphs. It fails against ooxml-common main.
  • ooxml-common>=0.13.0.

Merge order: merge uvrt/ooxml-common#24 first. Then revert the # TEMPORARY (gaps) CI pin in .github/workflows/ci.yml back to @main before merging this one.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WCxhY1fvWQPiWCwz7SYUVh

uvrt and others added 2 commits October 11, 2026 15:55
…oint does

empty_placeholders reports a placeholder holding only whitespace and warns of the stray
bullet PowerPoint draws for it; with ooxml-common 0.13.0 the render draws it too, and a
placeholder of empty paragraphs still renders nothing. CI installs ooxml-common from its
gaps-ws-bullet branch until uvrt/ooxml-common#24 is merged (TEMPORARY).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WCxhY1fvWQPiWCwz7SYUVh
@uvrt
uvrt merged commit 7f0ff40 into main Oct 13, 2026
19 checks passed
@uvrt
uvrt deleted the gaps-ws-bullet branch October 13, 2026 06:03
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