Krita's Crop Tool has a checkbox to keep ratio constant. But after either side of the cropping area touching the edges of the canvas, continue to drag it outward will change the ratio anyway. Reproducible: Always Steps to Reproduce: 1. Create a new document; 2. Crop Tool; 3. Tool Options >> Check [Keep Ratio Constant], value 1.00; 4. Drag the handles so one side of the cropping area touches the edge of canvas; 5. Continue to drag outward. Actual Results: The ratio changes even when Keep Ratio Constant is checked. Expected Results: The ratio does not change. You cannot make the cropping area any larger after one side touches the edge of canvas. See GIMP for reference. This bug was NOT triggered by the solutions of Bug 342842 , Bug 342844 and Bug 343287. It has been around for ages. I just forgot to report it every time. Sorry!
Created attachment 90820 [details] Crop Tool loses constant ratio after touching either edge of the canvas In this screenshot I was dragging the Crop Tool from bottom-right to top-left. I continue to drag outward after the handles touching the left side of the canvas. You can see Keep Ratio was checked, value set to 1.00, but the Crop Area is definitely not square.
Created attachment 91316 [details] I can confirm. The crop box doesn't respect the ratio when is dragged over the canvas edges. In my opinion, this behavior frustrates the point of having set a fixed ratio (video in attachment)
What would the preferred behaviour be? Don't shrink the box and simply stop once it reaches the crop border, or shrink the box but respect aspect ratio?
(In reply to Victor Wåhlström from comment #3) > What would the preferred behaviour be? Don't shrink the box and simply stop > once it reaches the crop border, or shrink the box but respect aspect ratio? Hello, personally I'd like it to be "Don't shrink the box and simply stop once it reaches the crop border".
+1 for Comment 4
Actually -- When "Grow" is unchecked: Stop enlarging crop area once the moving side touches the edge of the canvas. When "Grow" is checked: Allow enlarging crop area in any case but always respect ratio.
*** Bug 333369 has been marked as a duplicate of this bug. ***
Anyone who builds krita themselves, please test https://invent.kde.org/kde/krita/merge_requests/162
Git commit 9b45d78661e01f54f5bd78ae2233407ee79d0ea3 by Ivan Yossi. Committed on 04/10/2019 at 16:25. Pushed by ivany into branch 'master'. Fix Crop tool loosing aspect ratio on move Previously there was no check for boundaries after the new rect position is calculated. Before applying any offset on move drag we check we no longer go ouside of the image bounds. For aspect ratio we keep aspect ratio growing into the image instead of outwards getting some weird behaviour M +22 -0 plugins/tools/tool_crop/kis_constrained_rect.cpp https://invent.kde.org/kde/krita/commit/9b45d78661e01f54f5bd78ae2233407ee79d0ea3
Thank you Vanyossi! :D
Git commit 8d032ed4bd01eb2a28eaff42c78ec2d56e36b850 by Boudewijn Rempt, on behalf of Ivan Yossi. Committed on 09/10/2019 at 08:40. Pushed by rempt into branch 'krita/4.2'. Fix Crop tool loosing aspect ratio on move Previously there was no check for boundaries after the new rect position is calculated. Before applying any offset on move drag we check we no longer go ouside of the image bounds. For aspect ratio we keep aspect ratio growing into the image instead of outwards getting some weird behaviour (cherry picked from commit 9b45d78661e01f54f5bd78ae2233407ee79d0ea3) M +22 -0 plugins/tools/tool_crop/kis_constrained_rect.cpp https://invent.kde.org/kde/krita/commit/8d032ed4bd01eb2a28eaff42c78ec2d56e36b850
*** Bug 413334 has been marked as a duplicate of this bug. ***