Bug 378862 - When using move tool, the step (move action) is not recorded unless you activate any other tool.
Summary: When using move tool, the step (move action) is not recorded unless you activ...
Status: RESOLVED DUPLICATE of bug 392014
Alias: None
Product: krita
Classification: Applications
Component: Tools/Move (show other bugs)
Version: git master (please specify the git hash!)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-17 08:19 UTC by Raghavendra kamath
Modified: 2018-08-29 13:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
video demonstrating the bug (533.63 KB, video/mp4)
2017-04-17 08:19 UTC, Raghavendra kamath
Details
Kra file with layer for testing this bug (532.71 KB, application/x-krita)
2017-04-17 08:20 UTC, Raghavendra kamath
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Raghavendra kamath 2017-04-17 08:19:56 UTC
Created attachment 105054 [details]
video demonstrating the bug

I am using Krita built from git master (git 9D352f0) on arch linux machine
Qt version is 5.8

When I use move tool and make repeated move actions, the actions are not actually recorded in the undo history. The action is recorded only when I click on any other tool. And due to this only the last move action is recorded. You can't go back one move action at a time.

In my opinion each move action should be recorded. when user releases the left click button or releases the pen tablet tip after move.

I noticed this because , I changed a position of a layer in a document and I closed the document by mistake, instead of asking the modification (move action) to be saved krita closed as if the document is not modified at all, since I used only move tool and didn't switch to any other tool before closing my move action was gone and not recorded.

I am also attaching a video of the bug and also attaching a test file. Just try to move the layer with black box  while move tool is activea nd notice the undo history docker.
Comment 1 Raghavendra kamath 2017-04-17 08:20:31 UTC
Created attachment 105055 [details]
Kra file with layer for testing this bug
Comment 2 Halla Rempt 2017-06-05 08:47:55 UTC
I first thought that this was intentional, but there's

commit 8eb0cd01f8f844a02432088f3c963468b95046ad
Author: Dan Leinir Turthra Jensen <admin@leinir.dk>, Tue Feb 18 15:23:06 2014 +0000 (3 years, 4 months ago)
Committer: Boudewijn Rempt <boud@valdyas.org>, Wed Feb 19 10:10:33 2014 +0100 (3 years, 4 months ago)
Precedes: v2.99.88
Branches: <Expand>

Make each move using the move tool an undoable action.

This will increase the undo stack faster, but it means we get a
proper undo action for each move, without having to accept the
move in some secret way (enter will do it, or changing to anoher
tool). It also ensures we get the ability to undo move actions
in Sketch mode immediately.

BUG:330704
Comment 3 Halla Rempt 2018-08-29 13:36:01 UTC

*** This bug has been marked as a duplicate of bug 392014 ***