kicad/gerbview/tools
Jeff Young 77334628c4 Change default tool behaviour to skip unhandled events.
The problem is that wxEVT_CHAR_HOOK doesn’t do the key translation
properly.  wxEVT_CHAR does, but we only get to that if we skip the
event at the end of the tool’s event processing loop, which most tools
don’t do.  (Selection tools, point editors, pickers, and a couple of
others do skip, which is probably why this didn’t get reported earlier.)

I played around with a couple of ways to fix wxEVT_CHAR_HOOK.  Most of
them don’t work, and the few egregious hacks I tried weren't cross-
platform.

So I’m changing it so that most tools now skip at the end of their
event loops.  I left out a couple that I felt were high risk (length
tuning, for instance).  But there’s still enough risk that I’m 100%
sure it will break something, I just haven’t a clue what.

Fixes: lp:1836903
* https://bugs.launchpad.net/kicad/+bug/1836903
2019-07-26 12:21:24 -06:00
..
gerbview_actions.cpp Homogenize selection tools around our HIG. 2019-07-09 21:59:46 +01:00
gerbview_actions.h Move hotkeys to ACTION architecture. 2019-06-10 23:46:00 +01:00
gerbview_control.cpp Implement I18N for ACTIONs. 2019-06-17 10:45:23 +01:00
gerbview_control.h Move hotkeys to ACTION architecture. 2019-06-10 23:46:00 +01:00
gerbview_selection.cpp Convert separate-compilation of selection.cpp to polymorphism. 2019-06-10 23:46:00 +01:00
gerbview_selection.h Convert separate-compilation of selection.cpp to polymorphism. 2019-06-10 23:46:00 +01:00
gerbview_selection_tool.cpp Change default tool behaviour to skip unhandled events. 2019-07-26 12:21:24 -06:00
gerbview_selection_tool.h Cleanup and commenting. 2019-07-14 10:36:48 +01:00