Bug 386476

Summary: First mouse click after drawing with pen is ignored when using Pointer Input API
Product: [Applications] krita Reporter: Alvin Wong <alvin>
Component: Tablets (tablet issues are only very rarely bugs in Krita!)Assignee: Alvin Wong <alvin>
Status: RESOLVED FIXED    
Severity: normal CC: halla
Priority: NOR    
Version: git master (please specify the git hash!)   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: Version Fixed In:

Description Alvin Wong 2017-11-02 18:59:29 UTC
It might be due to it not sending mouse down/up events when pen events are accepted... Not sure if I should have it always send mouse events or try to fix KisInputManager? (But any changes to KisInputManager might break some other tablet's WinTab handling...)
Comment 1 Halla Rempt 2017-12-12 07:58:40 UTC
reminds me of https://bugs.kde.org/show_bug.cgi?id=387807 ...
Comment 2 Alvin Wong 2017-12-12 15:30:52 UTC
Nah, it's different, even though both issues are somehow related to the mouse event blocking function.
Comment 3 Alvin Wong 2017-12-12 15:56:46 UTC
Git commit b0dbfeb13c368ce262a2e959fd1ffa0bca32f099 by Alvin Wong.
Committed on 12/12/2017 at 15:56.
Pushed by alvinwong into branch 'master'.

WinInk: Simulate native mouse events for handled pen events

The code now simulates native mouse events with `SetCursorPos` and
`SendInput` for pen pointer events whose QTabletEvents are handled by
the widgets, so that the cursor position is updated to get correct
results from QCursor::pos().
Related: bug 386475
Differential Revision: https://phabricator.kde.org/D8801

M  +102  -15   libs/ui/input/wintab/kis_tablet_support_win8.cpp

https://commits.kde.org/krita/b0dbfeb13c368ce262a2e959fd1ffa0bca32f099
Comment 4 Alvin Wong 2017-12-12 16:19:40 UTC
Git commit 2ae618848b4fd9ca76bc71abd41a9f41fb328d5e by Alvin Wong.
Committed on 12/12/2017 at 16:04.
Pushed by alvinwong into branch 'krita/3.3'.

WinInk: Simulate native mouse events for handled pen events

The code now simulates native mouse events with `SetCursorPos` and
`SendInput` for pen pointer events whose QTabletEvents are handled by
the widgets, so that the cursor position is updated to get correct
results from QCursor::pos().
Related: bug 386475
Differential Revision: https://phabricator.kde.org/D8801

(cherry picked from commit b0dbfeb13c368ce262a2e959fd1ffa0bca32f099)

M  +102  -15   libs/ui/input/wintab/kis_tablet_support_win8.cpp

https://commits.kde.org/krita/2ae618848b4fd9ca76bc71abd41a9f41fb328d5e