Bug 119038 - Fileopen DOCX: images outside table
Summary: Fileopen DOCX: images outside table
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium normal
Assignee: Attila Bakos (NISZ)
URL:
Whiteboard: target:7.0.0
Keywords: filter:docx
Depends on:
Blocks: DOCX-Tables layoutInCell
  Show dependency treegraph
 
Reported: 2018-08-01 13:19 UTC by Telesto
Modified: 2024-08-26 10:17 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file (35.55 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2018-08-01 13:20 UTC, Telesto
Details
comparison MSO 2010 and LibreOffice 6.2 (292.32 KB, image/png)
2018-08-01 19:50 UTC, Xisco Faulí
Details
The Sample File After Fix (264.18 KB, image/png)
2020-04-08 10:05 UTC, Attila Bakos (NISZ)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2018-08-01 13:19:46 UTC
Description:
Fileopen DOCX: images outside table

Steps to Reproduce:
1. Open the attached file
2. Image should be inside the table, but aren't 

Actual Results:
Images floating above the tables

Expected Results:
Image should be inside the tables


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 6.2.0.0.alpha0+
Build ID: 1b21ff86effe58ae368457de8fec654ba4c8edd9
CPU threads: 4; OS: Windows 6.3; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2018-07-30_03:13:35
Locale: nl-NL (nl_NL); Calc: CL

and in
6.0

No images in
Version: 5.4.1.0.0+
Build ID: f200d5700782ae179fd96b6ad4b0fe8e7edd1616
CPU threads: 4; OS: Windows 6.29; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL
Comment 1 Telesto 2018-08-01 13:20:03 UTC
Created attachment 143889 [details]
Example file
Comment 2 Xisco Faulí 2018-08-01 19:50:10 UTC
Created attachment 143914 [details]
comparison MSO 2010 and LibreOffice 6.2
Comment 3 Xisco Faulí 2018-08-01 19:53:33 UTC
Reproduced in

Version: 5.2.0.0.alpha0+
Build ID: 3ca42d8d51174010d5e8a32b96e9b4c0b3730a53
Threads 4; Ver: 4.10; Render: default; 

Version: 4.3.0.0.alpha1+
Build ID: c15927f20d4727c3b8de68497b6949e72f9e6e9e

Version 4.1.0.0.alpha0+ (Build ID: efca6f15609322f62a35619619a6d5fe5c9bd5a)

LibreOffice 3.3.0 
OOO330m19 (Build:6)
tag libreoffice-3.3.0.4

@Justin Luth, I thought you could be interested in this issue...
Comment 4 QA Administrators 2019-08-03 03:06:33 UTC Comment hidden (obsolete, spam)
Comment 5 Justin L 2019-08-13 09:49:53 UTC
and as an exercise in mirth, use LO to round-trip the file and everything will be reversed. Now LO looks like how the original was expected to look, and Office 2003 looks like the broken LO.
Comment 6 Justin L 2019-08-14 08:08:10 UTC
(In reply to Justin L from comment #5)
> and as an exercise in mirth, ... Office 2003 looks like the broken LO.
Not true with Office 2016 though. 2016 looks fine the whole time.

The big difference in the roundtrip is that initially LO imports the circles as draw shapes, and the round-trip converts them into images. Apparently LO natively handles the area taken up by shapes very differently from images.

Since round-tripping this already looks VERY different in LO, and it also "fixes" the problem, I'll just leave this bug as is. Occasionally there is talk of hiring someone to fix image handling - this is probably a good bug to include in that workload, since it highlights some of the fundamental differences.
Comment 7 Xisco Faulí 2019-10-21 09:46:08 UTC
Still reproducible in

Version: 6.4.0.0.alpha1+
Build ID: de4839e66d3d195315729b95cc144cdab96b6e74
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded
Comment 8 Commit Notification 2020-04-07 10:45:51 UTC
Attila Bakos committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/27d04f6dbf38aa28fb7215590d578c4567db5770

tdf#119038 DOCX: fix FollowTextFlow handling

It will be available in 7.0.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.
Comment 9 Attila Bakos (NISZ) 2020-04-08 10:05:37 UTC
Created attachment 159422 [details]
The Sample File After Fix
Comment 10 BogdanB 2020-05-02 20:59:06 UTC
Verified in
Version: 7.0.0.0.alpha0+
Build ID: 05b87ae66ebccd06995cf530b860443ad9503b31
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3; 
Locale: ro-RO (ro_RO.UTF-8); UI-Language: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2020-05-02_10:14:58
Calc: threaded