Skip to content

EMF image support #196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

eruffaldi
Copy link

This patch allows to add EMF images. EMF contains size in 0.01 mm units, so I have picked 300 DPI for computing pixel size.

Provided also a sample script with EMF for testing

@JustMe85
Copy link

As you described it it doesn't work for me. I had to change the signature offset (image.__init__.py) to 41 instead of 40. After this minor modification it seems to work just fine. Great addition, thanks!

@eruffaldi
Copy link
Author

@JustMe85 can we Merge it?

@malcolmroberts
Copy link

This would be really great to merge; any issues?

@JustMe85
Copy link

JustMe85 commented Nov 9, 2019

No issues That I know of

@malcolmroberts
Copy link

@scanny ? Are you active on this project?

aleksandarbos added a commit to openlawlibrary/python-docx that referenced this pull request Feb 14, 2020
- implemented WMF support for python-docx based on existing PR
python-openxml#196
@spillz
Copy link

spillz commented Jan 12, 2021

Anyone waiting in vain on this and needs EMF support without having to handroll their own patched version of the docx lib can try this monkey patch version of the PR code:

https://gist.github.com/spillz/1667dd8b04654f32b51133cb7f72b898

@constructor-s
Copy link

Anyone waiting in vain on this and needs EMF support without having to handroll their own patched version of the docx lib can try this monkey patch version of the PR code:

https://gist.github.com/spillz/1667dd8b04654f32b51133cb7f72b898

Works wonderfully so far! Thank you. Hope this can one day be added without patching.

@100apm
Copy link

100apm commented Oct 10, 2021

it works with me ,thx a lot!

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.

6 participants