Review Request 124278: Improve Krita input device switching for Surface Pro 3

Michael Abrahams miabraha at gmail.com
Tue Jul 7 16:33:11 BST 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124278/
-----------------------------------------------------------

(Updated July 7, 2015, 3:33 p.m.)


Review request for Calligra.


Changes
-------

Split patch into sections.  (1/3)


Bugs: 341899
    http://bugs.kde.org/show_bug.cgi?id=341899


Repository: calligra


Description
-------

This patch rewrites most of kis_tablet_support_win.cpp to support basic
use of the eraser key on the Surface Pro 3.  The basic issue is that the
SP3 eraser button does not register being held until the stylus is
touched to the screen. A secondary issue is that the eraser button might
be pressed in the middle of the stroke, disrupting the ordinary
schedule of press and release events.

This patch attempts to handle those issues by watching for "inline"
cursor changes.  If a packet pops up with a different cursor ID, we will
dispatch a release event and a tool switch signal.  This breaks some
layers of abstraction but it seems to work.

The rest of the rewrite mostly focuses on readability, but adds a few
assorted fixes elsewhere, in particular allowing the current tool to be
changed by hovering over the dockers (necessary to allow the SP3 to
set an alternate alternate tool with the eraser button), an addition to
the x11 tablet code to give the same functionality, a prescaling feature
for QTabletEvent, preventing multiple instances of the Qt/Wintab dialog,
and a few typo corrections.


Diffs (updated)
-----

  krita/ui/input/wintab/kis_screen_size_choice_dialog.cpp 364419b 
  krita/ui/input/wintab/kis_tablet_support.h 8c1b279 
  krita/ui/input/wintab/kis_tablet_support_win.cpp 5e5d82f 
  libs/flake/KoToolManager.cpp 731faed 

Diff: https://git.reviewboard.kde.org/r/124278/diff/


Testing
-------


Thanks,

Michael Abrahams

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150707/f15071ed/attachment.htm>


More information about the calligra-devel mailing list