| Summary: | FILEOPEN DOCX Text boxes slip into table | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Aron Budea <aron.budea> |
| Component: | filters and storage | Assignee: | Not Assigned <libreoffice-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | arato.daniel, bakos.attilakaroly, libreoffice, nemeth, telesto, timar74 |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 7.1.0.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: |
https://bugs.documentfoundation.org/show_bug.cgi?id=135198 https://bugs.documentfoundation.org/show_bug.cgi?id=119038 https://bugs.documentfoundation.org/show_bug.cgi?id=139661 |
||
| Whiteboard: | target:7.4.0 target:7.3.1 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 104449, 139532 | ||
| Attachments: |
Sample DOCX
Screenshot from Word Screenshot from Writer in 7.3 |
||
|
Description
Aron Budea
2021-08-24 07:06:25 UTC
Created attachment 174507 [details]
Screenshot from Word
Created attachment 174508 [details]
Screenshot from Writer in 7.3
The commit before c9eb53f20022, the one found with bisect was ebea072fa7d7. Using ebea072fa7d7, you ostensibly get the text boxes positioned correctly, but if you try to select them you will find that they are detached from the frames they are supposed to be in. All c9eb53f20022 did was make this problem more apparent. So the real problem is the frames themselves being stuck inside the table, not the text boxes per se. Good catch, thanks for pointing that out! Turns out that's a regression as well, and could be bibisected to the following commit using repo bibisect-linux-64-7.0. https://cgit.freedesktop.org/libreoffice/core/commit/?id=27d04f6dbf38aa28fb7215590d578c4567db5770 author Attila Bakos <bakos.attilakaroly@nisz.hu> 2020-03-19 14:40:36 +0100 committer László Németh <nemeth@numbertext.org> 2020-04-07 12:44:17 +0200 tdf#119038 DOCX: fix FollowTextFlow handling (In reply to Aron Budea from comment #4) > Good catch, thanks for pointing that out! Turns out that's a regression as > well, and could be bibisected to the following commit using repo > bibisect-linux-64-7.0. > > https://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=27d04f6dbf38aa28fb7215590d578c4567db5770 > author Attila Bakos <bakos.attilakaroly@nisz.hu> 2020-03-19 14:40:36 +0100 > committer László Németh <nemeth@numbertext.org> 2020-04-07 12:44:17 +0200 > > tdf#119038 DOCX: fix FollowTextFlow handling After I fix the Group Shape Text Box supporting tickets, I will fix this one as well. Unfortunately, is seems to me the LayoutInCell option in Word not really equals to FollowTextFlow, as that commit maps that. Solution might be introducing a new function something like LayoutInCell to Writer, or introducing a new compat-flag which modifies FllowTextFlow in that cases like this. Fix coming soon, and Adding CC: to Attila Bakos (In reply to Attila Bakos (NISZ) from comment #5) > (In reply to Aron Budea from comment #4) > > Good catch, thanks for pointing that out! Turns out that's a regression as > > well, and could be bibisected to the following commit using repo > > bibisect-linux-64-7.0. > > > > https://cgit.freedesktop.org/libreoffice/core/commit/ > > ?id=27d04f6dbf38aa28fb7215590d578c4567db5770 > > author Attila Bakos <bakos.attilakaroly@nisz.hu> 2020-03-19 14:40:36 +0100 > > committer László Németh <nemeth@numbertext.org> 2020-04-07 12:44:17 +0200 > > > > tdf#119038 DOCX: fix FollowTextFlow handling > > After I fix the Group Shape Text Box supporting tickets, I will fix this one > as well. Unfortunately, is seems to me the LayoutInCell option in Word not > really equals to FollowTextFlow, as that commit maps that. > Solution might be introducing a new function something like LayoutInCell to > Writer, or introducing a new compat-flag which modifies FllowTextFlow in > that cases like this. > Fix coming soon, and > Adding CC: to Attila Bakos Group Shapes are ready, so fix is there: https://gerrit.libreoffice.org/c/core/+/128413 tdf#144045 tdf#142484 Limit off-page positioning to textboxes For the record Miklos had two fixes related to this that make the sample DOCX look nice already: https://cgit.freedesktop.org/libreoffice/core/commit/?id=3eb6d764b3023500f2299d36bf1860bc8e67db9f https://cgit.freedesktop.org/libreoffice/core/commit/?id=e993638d5ecd33783f2eebdccfa87a81e5a8a2c5 Of course further related fixes/improvements awe always welcome. (In reply to Aron Budea from comment #7) > For the record Miklos had two fixes related to this that make the sample > DOCX look nice already: > https://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=3eb6d764b3023500f2299d36bf1860bc8e67db9f > https://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=e993638d5ecd33783f2eebdccfa87a81e5a8a2c5 > > Of course further related fixes/improvements awe always welcome. You are right, i did not known about these fixes and make a new one. Only when i finished the test that time i recognized, the fix is independent from me. :) Should i close this ticket as fixed? Yes, I think we can, thank you! |