Bug 115562 - Page anchored rotated image has wrong movement for arrow keys
Summary: Page anchored rotated image has wrong movement for arrow keys
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.1.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Images
  Show dependency treegraph
 
Reported: 2018-02-08 19:54 UTC by Regina Henschel
Modified: 2019-02-22 10:36 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Regina Henschel 2018-02-08 19:54:42 UTC
Start with a new text document and insert an image. Anchor it to page. Test that the image moves correctly, if you use the arrow keys. Rotate the image e.g. by 45 degree. Test the arrow keys again. The movement of the image does not follow the arrow keys.
Comment 1 raal 2018-02-08 21:22:16 UTC
I can confirm with Version: 6.1.0.0.alpha0+
Build ID: 30083601334279f33a3021dfe7d22970d02c8e9e
CPU threads: 4; OS: Linux 4.4; UI render: default; VCL: gtk3;
Comment 2 V Stuart Foote 2018-02-08 22:07:35 UTC
Where in the GUI? Which Arrow keys?

Rotation on the Sidebar Properties Deck -> Position and Size content panel's drop list box moves up or down the list rotating image to match.  The 360 deg rotation medallion there controls the image.

Rotation on the Image -> Properties dialog Image tab? In the Rotation Angle area, arrows on the spin box control rotates the image (if slowly 1/100 deg steps) when applied. While clicking on a 15 deg index mark (or dragging) the 360 deg rotation medallion works when applied.

Aren't <Alt>+[UP,DONW,LEFT,RIGHT] controls to rotate Draw objects? And, are they still working at 6.0--they seem to now be fine positioning controls (even in Rotation mode). With <Shift>+[UP,DOWN,LEFT,RIGHT] acting as coarse movement controls.
Comment 3 Regina Henschel 2018-02-08 23:36:41 UTC
(In reply to V Stuart Foote from comment #2)
> Where in the GUI? Which Arrow keys?

The UP, RIGHT, DOWN, LEFT keys on the keyboard. If a shape is selected, they allow movement of the shape. That works with unrotated images the same, but not for rotated images.

> 
> Aren't <Alt>+[UP,DONW,LEFT,RIGHT] controls to rotate Draw objects?

No. It is fine move by pixel.

 And, are
> they still working at 6.0--they seem to now be fine positioning controls
> (even in Rotation mode).

As it always has been.

 With <Shift>+[UP,DOWN,LEFT,RIGHT] acting as coarse
> movement controls.

That has not changed too for shapes and unrotated images, but does not work for rotated images.
Comment 4 V Stuart Foote 2018-02-09 00:06:36 UTC
(In reply to Regina Henschel from comment #3)

I am sorry I misread things. But now I follow. It is the _movement_ of the Page anchored image, not its rotation that is wonky.

But yes, when anchored to Page (not with defaults anchored to Paragraph) movement of the "rotated" image does not track with the <Alt>+[UP,DONW,LEFT,RIGHT] or <Shift>+[UP,DONW,LEFT,RIGHT] or [UP,DONW,LEFT,RIGHT] movment commands.

Anchored to Paragraph movement seems correct.

=-testing-=
Windows 10 Ent 64-bit en-US with
Version: 6.1.0.0.alpha0+ (x64)
Build ID: 609888f3c8d6c0fe72c41ac26de431a12ad3fdd0
CPU threads: 8; OS: Windows 10.0; UI render: GL; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2018-02-08_02:03:29
Locale: en-US (en_US); Calc: group
Comment 5 raal 2018-02-09 09:42:08 UTC
This seems to have begun at the below commit.

c14b853906570a66fb8e5b505b88ed91aae96b00 is the first bad commit
commit c14b853906570a66fb8e5b505b88ed91aae96b00
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Tue Dec 12 21:28:31 2017 -0800

    source sha:6675e6eaf999de94d49d7644d5877537fda83239

author	Armin Le Grand <Armin.Le.Grand@cib.de>	2017-11-02 18:28:04 +0100
committer	Armin Le Grand <Armin.Le.Grand@cib.de>	2017-11-18 13:07:43 +0100
commit	6675e6eaf999de94d49d7644d5877537fda83239 (patch)
tree	588b33264a822271d2840d7d9c39dc31212f642e
parent	995c75de956436606e4f313d60257285955586d8 (diff)
RotateFlyFrame3: Initial support added
First steps to get a rotated FlyFrame and content
that only uses layouted sizes/positions and does not
change the model data (except rotation). This works
with persistence, after reload the rotation can be
resetted with going back to the original FrameSize.
Lot of stuff not yet working, experimental state.
Comment 6 Xisco Faulí 2018-02-28 14:59:05 UTC
Adding Cc: to Armin Le Grand