Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Oct 21, 2022. It is now read-only.

Conversation

@drewfulton
Copy link

@drewfulton drewfulton commented Feb 18, 2022

There is a lot going on here. I'll try to give a basic run down but I honestly don't have a completely understanding of everything.

Smart Art objects are held in a Graphic Frame just like Charts and Tables. Like Charts, the smart art contents are stored in a different, related xml file. Unlike charts, this is actually 5 different XML files. 4 of those are referenced directly by the slide while the 5th (diagram.xml) is referenced from within the data.xml file.

My basic approach here was to get enough implementation that I could basically dump each of those 5 files into a string and then reproduce them with the batch tester. Ultimately, it worked pretty smoothly once I got it all worked out.

The most complicated part was retrieving and updating the reference to diagram.xml from within data.xml but I eventually got it.

This is also built so that it can work with D2-523 which makes it work with the slide importer.

@drewfulton
Copy link
Author

Just as a heads up, I ran a medium batch on our current Deckbot develop branch using this branch of python-pptx and we are good to release this without requiring a deckbot release. They can be done independently, this just needs to be done before the SmartArt component of the importer.

@drewfulton drewfulton merged commit 71f0d1c into develop Feb 23, 2022
@drewfulton drewfulton deleted the feature/D2-543 branch February 23, 2022 15:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.