| Summary: | Wrong horizontal position for the RTL content control fields in the exported PDF file | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Hossein <hossein> |
| Component: | Writer | Assignee: | Hossein <hossein> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | vmiklos |
| Priority: | medium | ||
| Version: | 7.4.0.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:7.6.0 target:7.5.4 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 107742 | ||
| Attachments: |
PDF output from LO 7.5 dev master
Wrong hotizontal position of the fields visible in the PDF file opened in Firefox |
||
|
Description
Hossein
2022-11-26 21:12:17 UTC
@Miklos: I would like to fix this issue. Could you please give me some advise/code pointers? I see boundary calculations inside SwContentControlPortion::DescribePDFControl in sw/source/core/text/itrform2.cxx. Created attachment 183842 [details]
Wrong hotizontal position of the fields visible in the PDF file opened in Firefox
This is a screenshot of the bad positioning of the fields in the PDF file, visible when opened in Firefox. It is worth mentioning that even the positions of the fields in the last line that seem to be OK are wrong. That is because the place of month / year is reversed, suggesting that the calculation of the positions should be somehow reversed for RTL.
Yes, all content-control-specific PDF export code is in SwContentControlPortion::DescribePDFControl(), you found the correct place already. :-) Thanks for looking into this! Hossein committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/1466e2a58f40c869d8c2590a4479e85c093b3468 tdf#152246 Fix RTL content control field position It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Hossein committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/57f46a5a64f4e3bca2f0a52309ca7d201c1e2372 tdf#152246 Test position of RTL elements in PDF It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Hossein committed a patch related to this issue. It has been pushed to "libreoffice-7-5": https://git.libreoffice.org/core/commit/df461b42126e440492fc919d2b1b2d6649962a90 tdf#152246 Fix RTL content control field position It will be available in 7.5.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |