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

Pedro <pe...@outlook.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/grap
                   |                            |hics/gwenview/-/commit/ee17
                   |                            |9a5de052a2e0502285ff53a4255
                   |                            |c5c9dd094
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #16 from Pedro <pe...@outlook.com> ---
Git commit ee179a5de052a2e0502285ff53a4255c5c9dd094 by Pedro Hernandez.
Committed on 02/05/2025 at 02:48.
Pushed by merritt into branch 'master'.

Fix crash when crop area is smaller than handles

The crash occurs due to a failed assertion in `qBound()` that tests
`Q_ASSERT(!(max < min))`. This happens when computing the position
of the crop area middle handles (top center, bottom center, left
 center, right center) and they overlap with the corner handles.

The fix proposed is to check if there is enough room within the
respective axis (width or height) for a middle handle to be
repositioned when the crop area is not completely inside the
viewport. This preserves the existing functionality added in
commits `307816e2819f` and `e61ce1e2886c6`.

M  +10   -4    lib/crop/croptool.cpp

https://invent.kde.org/graphics/gwenview/-/commit/ee179a5de052a2e0502285ff53a4255c5c9dd094

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

Reply via email to