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

Henrik Fehlauer <rk...@lab12.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/gwe
                   |                            |nview/e97eda1df871cebc87834
                   |                            |822a08c5f34fc80938b
   Version Fixed In|                            |18.08.0

--- Comment #4 from Henrik Fehlauer <rk...@lab12.net> ---
Git commit e97eda1df871cebc87834822a08c5f34fc80938b by Henrik Fehlauer.
Committed on 26/07/2018 at 18:36.
Pushed by rkflx into branch 'Applications/18.08'.

Fix crosshair cursor not showing in red eye reduction tool

Summary:
`RedEyeReductionTool::toolActivated` sets `Qt::CrossCursor` to allow for
more precise positioning where to apply the red eye reduction.

Due to `RasterImageView::setCurrentTool` calling `updateCursor`
afterwards, that change is immediately overridden again, though.

By moving around the latter, the issue can be fixed.
FIXED-IN: 18.08.0

Test Plan:
In {nav View} mode, activate the {nav Red Eye Reduction} tool. A
crosshair cursor should show, and hide again upon closing the tool.

Reviewers: #gwenview, ngraham

Reviewed By: #gwenview, ngraham

Subscribers: muhlenpfordt, ngraham

Differential Revision: https://phabricator.kde.org/D14386

M  +5    -1    lib/documentview/rasterimageview.cpp

https://commits.kde.org/gwenview/e97eda1df871cebc87834822a08c5f34fc80938b

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

Reply via email to