Bug 149247

Summary:	Image is overlapping text after rotating 90 degrees left/right (not honoring wrap)
Product:	LibreOffice	Reporter:	Telesto <telesto>
Component:	Writer	Assignee:	Not Assigned <libreoffice-bugs>
Status:	NEW ---
Severity:	normal	CC:	Armin.Le.Grand, buzea.bogdan, kelemeng, raal
Priority:	medium	Keywords:	bibisected, bisected, regression
Version:	6.1.0.3 release
Hardware:	All
OS:	All
Whiteboard:
Crash report or crash signature:		Regression By:	Armin Le Grand
Bug Depends on:
Bug Blocks:	147763
Attachments:	Example file Screenshots showing the problem

Description Telesto 2022-05-23 17:48:49 UTC

Description:
Image is overlapping text after rotating 90 degrees left/right (not honoring wrap)

Steps to Reproduce:
1. Open the attached file
2. Select the cat image
3. Press rotate 90 degree in context menu or toolbar

Actual Results:
Cat image is overlapping part of the text

Expected Results:
Shouldn't be the case


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
7.4

and in
Version: 7.0.7.0.0+ (x64)
Build ID: 626ea4e62a3e5005fe9825923a1c0c5bdb61cc08
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

Comment 1 Telesto 2022-05-23 17:49:02 UTC

Created attachment 180319 [details]
Example file

Comment 2 Telesto 2022-05-23 17:53:23 UTC

Also in
Version: 6.1.0.0.alpha0+
Build ID: e1b247a843c5eb850fe0079819239d9883412d38
CPU threads: 4; OS: Windows 6.3; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL

still OK with
Version: 6.0.0.0.alpha0+
Build ID: 9127d1a89cbfba89eb9df6755ea7b9e161cfc67a
CPU threads: 4; OS: Windows 6.3; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL

Comment 3 Rafael Lima 2022-05-23 22:43:00 UTC

Created attachment 180321 [details]
Screenshots showing the problem

I can confirm this bug using LO 7.3. See attached image showing that the image indeed overlaps with text if you change its rotation to 90º.

However, if you first rotate to 45º and then to 90º it works. It shouldn't have to need this to work, so setting this to NEW.

System info:
Version: 7.3.3.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 12; OS: Linux 5.13; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.3~rc2-0ubuntu0.21.10.1~lo1
Calc: threaded

Comment 4 Rafael Lima 2022-05-24 16:03:15 UTC

Hey sorry. I accidentally assigned this issue to myself. Setting to default.

Comment 5 raal 2023-05-15 15:37:30 UTC

This seems to have begun at the below commit in bibisect repository/OS 
/bibisect-win32-6.0.
Adding Cc: to 	Armin Le Grand ; Could you possibly take a look at this one?
Thanks
 c14b853906570a66fb8e5b505b88ed91aae96b00 is the first bad commit
commit c14b853906570a66fb8e5b505b88ed91aae96b00
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Tue Dec 12 21:28:31 2017 -0800

    source 6675e6eaf999de94d49d7644d5877537fda83239

https://git.libreoffice.org/core/+/6675e6eaf999de94d49d7644d5877537fda83239
RotateFlyFrame3: Initial support added