Created attachment 171997 [details] Screenshot of 6.3 vs 7.2 In 6.4+ images with contours are no longer correctly imported in DOC format. Steps to reproduce: 1. Open attachment 114438 [details] 2. Compare to LO 6.3 or Word
Created attachment 172000 [details] Screenshot of 6.3 vs 7.2
Created attachment 172001 [details] Screenshot of 6.3 vs 7.2
Note: this can be fixed by right clicking on the image: Wrap -> Contour
I guess duplicate of bug 140508, please see.
Same commit 8ce4782b86611dd481f89e4bffac61036cc7428e. *** This bug has been marked as a duplicate of bug 140508 ***
Not an exact duplicate. The other bug is from a different frame type (a textbox) while this one exactly matches the type of graphic (SwFlyObjDrawIdentifier). Well, I guess they still could be duplicates ultimately, but a partial fix would be to simply allow certain types to auto-contour. Plus, this one clearly is a regression. CC: Miklos for 6.4 (backported to 6.3.1) regression https://gerrit.libreoffice.org/c/core/+/76566/ commit 90928f23994d8fc1d9cc7edf95880782c8ca712a Author: Miklos Vajna on Mon Jul 29 20:49:21 2019 +0200 tdf#112535 DOC import: fix handling the have contour but no polygon case
Created attachment 187608 [details] WrappingBug.pdf: how it looks in Word 2010 - notice that the text is also wrapped around automatically
Removing regression, because I'm not sure there is a pin-pointable person. As noted in comment 7, the text OUGHT TO be wrapping around the years and other text in the image. Confirmed edit/auto contour defined (but no visible image at all) in OOo 3.3. The image became visible (with contour wrapped quite well) in 4.4 commit 6d431ffb682d0e64b75b6267f369822ff0b0617e Author: Michael Stahl on Sat May 24 18:34:06 2014 +0200 fdo#77454: fix WW8 import/export of negative image crop The contour wrapping got worse (years/text no longer included) in 6.2 with commit d4442ac1ac9aae36dbc08fda8154d71ea0f81708 Author: Noel Grandin on Wed May 30 13:02:25 2018 +0200 drop Graphic::GetBitmap The poor wrapping was "fixed" by Miklos' commit mentioned in comment 6. Currently, when contour is turned on then an auto-contour picks up most of the years/text - although not as good as before. The algorithm got worse after 7.2 commit 9d89d98d3349502b56da4bdd6ea287ac4cde9ce5 Author: Luboš Luňák on Wed Feb 10 17:15:19 2021 +0100 always optimize bitmap transform to translate+scale if possible (tdf#138068) Wow - what conclusion can be drawn from this?
Created attachment 187609 [details] WrappingBug.docx: round-tripped by MS Word 2010 - has contour-wrapped image I tried to answer the question "Does the autoshape have defined contour points, or just an auto-contour". The DOCX does have defined wp:wrapPolygon, suggesting yes. Unfortunately, LO doesn't contour wrap the DOCX version either.
Created attachment 187610 [details] 142284_dump.doc: doc_dump looking for relevant contour SPRMs For DOC format, it sounds like I need to be looking for a pWrapPolygonVertices property, which is an OfficeArtFOPTEOPID with an opid of 0x383. Since the opid is the lowest 14 bits, I should be able to find a 0xX383 number in the dump, but I don't see any.