-
Notifications
You must be signed in to change notification settings - Fork 399
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
Fix docx header issue #2026 #2068
base: master
Are you sure you want to change the base?
Fix docx header issue #2026 #2068
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very small changes requested
...itter.wpml/src/org/eclipse/birt/report/engine/emitter/wpml/writer/AbstractWordXmlWriter.java
Show resolved
Hide resolved
...itter.wpml/src/org/eclipse/birt/report/engine/emitter/wpml/writer/AbstractWordXmlWriter.java
Show resolved
Hide resolved
...t.engine.emitter.docx/src/org/eclipse/birt/report/engine/emitter/docx/writer/DocxWriter.java
Outdated
Show resolved
Hide resolved
The solution looks good, I understand your way. Sometimes I think I'm deep on it but you are deeper (much deeper). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good change
The showHeaderOnFirst property did not work for the DOCX emitter when using the combined RunAndRenderTask.
That is, the header was shown on the first page even when the property was false).
This PR makes this work.
It works with RunAndRenderTask as well as with separate RunTask and RenderTask.
The related property showFooterOnLast does not work with RunAndRenderTask,
only with separate RunTask and RenderTask.