[ankit2235]

A slide hidden in PowerPoint (<p:sld show="0">, Hide Slide; Skip slide in Google Slides) was converted like a normal slide, so its text went into the body and the default exports, although it is not presented.

The content of a hidden slide now goes to ContentLayer.INVISIBLE, the same as hidden Excel and ODS sheets (#1876): it stays available to callers who request that layer, and the default exports leave it out. Speaker notes and comments keep ContentLayer.NOTES. Visible slides are unchanged.

With the Google Slides file attached to #4646, the Markdown export now contains only slide 1; the hidden slide 2 is in the invisible layer.

The new test builds a deck with one visible and one hidden slide. It checks the default Markdown export, the invisible layer and the notes layer. It fails on main and passes with this change; the other PowerPoint tests pass unchanged.

If you prefer a different behavior for hidden slides (for example an option), I'm happy to change it.

Issue resolved by this Pull Request:
Resolves #4646

Checklist:

• Documentation has been updated, if necessary. (not needed)
• Examples have been added, if necessary. (not needed)
• Tests have been added, if necessary.

[github-actions]

✅ DCO Check Passed

Thanks @ankit2235, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[DanielNg0729]

Hi @ankit2235, can you resolve the merge conflict?

[github-actions]

AI triage (advisory)

Checked commit 237e82da90. The analysis read the diff only. It did not run code from this pull request.

The PR adds _hide_slide() to the PPTX backend. It moves all ContentLayer.BODY items in a hidden slide (attribute show="0") to ContentLayer.INVISIBLE, matching the existing behavior for hidden Excel sheets. A new test builds a deck with one visible and one hidden slide and checks the Markdown output, the invisible layer, and the notes layer.

Topic labels added: bug, pptx.

CI safety: no concern found. A maintainer must still approve CI.

2 related pull request(s) in the same code

PR	State	Signal	Assessment
#4443	open	changes the same code	related: PR 4443 refactors the same _handle_text_elements path to add run-level formatting and hyperlinks; it touches many of the same lines but fixes a different problem and does not implement hidden-slide handling.
#1876	merged	referenced as #1876	related: PR 1876 is the merged Excel analog: it sets ContentLayer.INVISIBLE for hidden sheets. This PR applies the same pattern to PPTX, so they are related by design but solve different format-specific problems.

_{The ai:* labels are advisory. They do not approve CI and they do not add tests:full. A new push runs this check again.}

[ankit2235]

Hi @DanielNg0729
Done, merged the latest main and resolved the conflict in tests/test_backend_pptx.py (kept both tests). The PowerPoint tests pass.

[DanielNg0729]

Lgtm!