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

XML command #223

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

XML command #223

wants to merge 1 commit into from

Conversation

sarantoo
Copy link

I use literalXml alot with this workaround. It really useful in many circumstances. However, it adds extra 2 paragraphs before and after insertion point which is not ideal. So I came up with this additional XML command, that once inserted, replace existing paragraph the same way other commands did (LINK, IMAGE, HTML).

The literal XML need special treatment if it contains multiple root elements (fragment). I opt to put them under a root element and add them up to the xml node tree if the root presented.

@jjhbw
Copy link
Collaborator

jjhbw commented Aug 6, 2021

Sorry for the late reply. I'm usually a bit apprehensive about new functionality in this library, as stability is my primary concern at the moment. This looks nice though. Injecting formatting into reports using XML is still pretty hard, this would make it simpler. Can you perhaps add a few snapshot tests? These tests would also serve to showcase the functionality.

@FelixRelli
Copy link

Would be really nice to get this merged.

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.

None yet

3 participants