Skip to content
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

DocumentAssembler: Support for images #30

Open
wants to merge 8 commits into
base: vNext
Choose a base branch
from

Conversation

carzaza
Copy link

@carzaza carzaza commented Dec 17, 2020

This PR is a copy of the closed one in the original OfficeDev repository from @krembf (see PR#236).

This feature is important for us to be available as we are currently using a copy of that PR.

Added ability to embed image content in template. Input image can be supplied either as a base64 encoded string or as a path to file. Supports different layout options. For different samples, see unit tests.
Removed unused "Aspect" attribute
It is expected that Image Content Control will immediately follow ImageSelect element, thus containing paragraph as child element. There is a situation where some extra characters apppear before the Image Content Control thus causing a situation where Image Conent Control is inside the paragraph and not vise versa, as expected. In such case, the system should trigger an error.
- Added handling of the situation decrribed above
- Added unit test to verify proper behavior
- Added ability to use image content control inside header and footer
- Added unit tests to verify proper behavior
Modified document assembler to support # notation.
Added unit tests to verify proprer behavior. Tests which are having A suffix (e.g. DA270A-ImageSelect.docx) are duplicates of the original ones, but with # notation used instead.
Removed static variable _docPrId. Now updating images docPr ids at the end of the processing and not during the transform.
@VictorIranzo
Copy link

¿Any news related with this Pull Request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants