Created attachment 132354 [details] document Steps to reproduce: 1. Open attached document 2. Save it as DOCX 3. Open the new document Observed behaviour: all images are shrunk after roundtrip Reproduced in Version: 5.4.0.0.alpha0+ Build ID: 6152bf9ee9b2e348dee854921a5a5db1cfc72995 CPU threads: 4; OS: Linux 4.8; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group
Created attachment 132355 [details] comparison MSO 2010 and LibreOffice 5.4
Regression introduced by: author Justin Luth <justin_luth@sil.org> 2016-11-04 18:32:38 (GMT) committer Justin Luth <justin_luth@sil.org> 2016-11-07 04:09:41 (GMT) commit 8eff1decd91cbfb10094c25d4cf1d2b434a4da72 (patch) tree bcc51a30d8eaa4acbaff628dabff996c2273779f parent bc6017d66fcf37db1f632a3ce667f4c3bf9d1d02 (diff) tdf#41542 MSWordExport: accommodate image's borderless padding Honor the padding value even if the border is not visible. Previously the padding information was just thrown out, and the image scaled to fill the entire image size. The unit test for tdf#77454 needed to be tweaked because surprisingly it has border padding of 1 - which this patch turns into a negative crop. bisected with bibisect-linux-64-5.3 Adding Cc: to Justin Luth
Not really causing images to shrink, but losing large cropping values - showing more of the picture than was intended. Just a stupid mistake in choosing the wrong variable size. https://gerrit.libreoffice.org/36200 tdf#106974 docx export: Crop is "long", not sal_Int16
Justin Luth committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=20feed528a9b9d57b35805611a4279000a474c3d tdf#106974 docx export: Crop is "long", not sal_Int16 It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Justin Luth committed a patch related to this issue. It has been pushed to "libreoffice-5-3": http://cgit.freedesktop.org/libreoffice/core/commit/?id=dd3325f0938aeb79221a0bb765b0fbed0fef42e6&h=libreoffice-5-3 tdf#106974 docx export: Crop is "long", not sal_Int16 It will be available in 5.3.3. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.