SUMMARY STEPS TO REPRODUCE 1. Create a layer, draw something on it and add a transform mask to it (or open the test file from bug 455349) 2. Move the transform mask with the move tool OBSERVED RESULT SAFE ASSERT (krita): "mergeResult" in file C:/Packaging/workspace/Krita_Stable_Windows_Build/krita/libs/ui/tool/strokes/move_stroke_strategy.cpp, line 306 SOFTWARE/OS VERSIONS Windows: Windows 10 ADDITIONAL INFORMATION Nightly b846a63f2a (stable)
I forgot to say add a value keyframe from the animation curves docker in step 1...
Git commit aed62045caae675a38e5e9df0bb10bb05aac9284 by Eoin O'Neill. Committed on 29/06/2022 at 03:08. Pushed by eoinoneill into branch 'master'. Changed logic to prevent undo merge when moving an animated transform mask. Animation keyframe undos currently can't be merged, but it's not necessary currently and does not affect undo stack. Might be worth implementing better merging logic later though as it might smooth out performance long term. M +1 -3 libs/ui/tool/strokes/move_stroke_strategy.cpp https://invent.kde.org/graphics/krita/commit/aed62045caae675a38e5e9df0bb10bb05aac9284
Git commit 359a67e3532a859d9662d7ad824b3aae9aeb0654 by Eoin O'Neill. Committed on 29/06/2022 at 22:15. Pushed by eoinoneill into branch 'krita/5.1'. Changed logic to prevent undo merge when moving an animated transform mask. Animation keyframe undos currently can't be merged, but it's not necessary currently and does not affect undo stack. Might be worth implementing better merging logic later though as it might smooth out performance long term. (cherry picked from commit aed62045caae675a38e5e9df0bb10bb05aac9284) M +1 -3 libs/ui/tool/strokes/move_stroke_strategy.cpp https://invent.kde.org/graphics/krita/commit/359a67e3532a859d9662d7ad824b3aae9aeb0654