https://bugs.kde.org/show_bug.cgi?id=455349

Eoin O'Neill <eoinoneill1...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/grap
                   |                            |hics/krita/commit/4fbf6d6dc
                   |                            |2ed4dd54bf3edf2755e52d54082
                   |                            |516f
             Status|REPORTED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Eoin O'Neill <eoinoneill1...@gmail.com> ---
Git commit 4fbf6d6dc2ed4dd54bf3edf2755e52d54082516f by Eoin O'Neill.
Committed on 16/06/2022 at 03:17.
Pushed by eoinoneill into branch 'master'.

Animated Transforms now respect cropping translations.

NOTE: I'm not happy with how the translation is working as a whole with
transform masks after cropping. E.G. In Alvin's test image, the
circle drawn is roughly 512px units into the canvas. By that logic,
cropping the image should actually move the existing transform keys
to near 0 value. However, the actual translation is by roughly
3000 units. This looks correct visually, but the coordinates make
no sense.

I'll file this as a separate bug. It's not **majorly** important,
but it would be nice to have logical coordinates for a transformation
masks. I think this bug is there on the base transformation mask, but
isn't easily noticable until you have an animated transform mask on
where the translation values suddenly matter more.

M  +19   -0   
plugins/tools/tool_transform2/kis_animated_transform_parameters.cpp
M  +3    -0   
plugins/tools/tool_transform2/kis_animated_transform_parameters.h

https://invent.kde.org/graphics/krita/commit/4fbf6d6dc2ed4dd54bf3edf2755e52d54082516f

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to