-
Notifications
You must be signed in to change notification settings - Fork 397
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
DOCX: padding of element "Label" not implemented #1941
Comments
I'm not sure what you want to achieve. Maybe it would be better to just document that the DOCX emitter doesn't support padding for labels (and plain text dynamic text items?). |
The current elements of "dynamic text", "text", "data" supports the paddding values. My target is here to provide the padding based on the indent-mechanism for ms word of the "label"-element. |
I understand and that it will be useful in common cases. However, it will not fully support the box model (e.g. background color would not work as intentend, border will not work at all, I suppose) and it will change the behavior of existing reports. I was going to write: "Also, please keep in mind that items may use |
The enhancement is added with PR #1942 |
The current implementation of the DOCX emitter has no implementation of padding for the BIRT element "Label".
The padding of the label should be added according to margin to set the according indents.
( This enhancement is familiar with #1895)
The difference from "label" to dynamic text will be that no special grid will be used, only the indent will be used.
MS Word doesn't support a real padding.
Additional on it some methods of the existing margin-padding-calculation are doubled with multiple implementations.
I will provide a PR to add the padding-calculation and optimize the methods.
The text was updated successfully, but these errors were encountered: