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

Emmet O'Neill <emmetoneill....@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap
                   |hics/krita/commit/4fbf6d6dc |hics/krita/commit/ea817fb64
                   |2ed4dd54bf3edf2755e52d54082 |18c4866712ac88288113f8b6d2d
                   |516f                        |3b3e

--- Comment #3 from Emmet O'Neill <emmetoneill....@gmail.com> ---
Git commit ea817fb6418c4866712ac88288113f8b6d2d3b3e by Emmet O'Neill, on behalf
of Eoin O'Neill.
Committed on 21/06/2022 at 03:02.
Pushed by emmetoneill into branch 'krita/5.1'.

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/ea817fb6418c4866712ac88288113f8b6d2d3b3e

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

Reply via email to