Skip to content

Conversation

@aantich
Copy link

@aantich aantich commented Nov 9, 2025

  • Read pptx format
  • Parse "smart art" diagrams
  • smart list detection

.gitignore Outdated
Comment on lines 9 to 12
/test-docs
doc/pptx-reader-design-v2.md
doc/pptx-reader-design.md
doc/xlsx-reader-design.md
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These .gitignore changes are specific to your setup and probably shouldn't be included in the PR.

.gitignore Outdated
Comment on lines 53 to 55
doc/pptx-reader-design*.md
doc/xlsx-reader-design.md
test-docs/
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

@jgm
Copy link
Owner

jgm commented Nov 9, 2025

Thanks for the PR! I'm testing it locally, and here are two things I noticed.

Slide titles

% pandoc -f markdown -t pptx | pandoc -f pptx -t markdown
# Slide one

content

# Slide two

content two
^D
## Slide 1

Slide one

content

## Slide 2

Slide two

content two

Pandoc normally uses headings with the slide title, but these get replaced by the pptx reader with "Slide 1", "Slide 2" etc. Can this be fixed?

Missing p:sldIdList

When I generate pandoc's default reference.pptx and try to read it with this reader, I get:

Failed to parse PPTX: Missing p:sldIdLst

But the file opens just fine in PowerPoint; I think it is valid, hence that p:sldIdLst may be optional? Can we handle the case where it is missing?

@aantich
Copy link
Author

aantich commented Nov 9, 2025

thank you, on it

@jgm
Copy link
Owner

jgm commented Nov 9, 2025

In addition, the new format should be added to MANUAL.txt ; see the list under --readers.

@aantich
Copy link
Author

aantich commented Nov 9, 2025

Addressed all comments.

@jgm
Copy link
Owner

jgm commented Nov 12, 2025

Tests are failing - can you fix?

@jgm
Copy link
Owner

jgm commented Nov 19, 2025

Hi, just wanted to see if you're planning to finish polishing this up. I think the CI test failure is something trivial; this is close to being mergeable.

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.

2 participants