haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't skip handled events in GTK 

Now that our tool framework handles the hotkeys, we need to skip the
passed handling work-around for actions that are already handled in the
event.

Fixes: lp:1832604
* https://bugs.launchpad.net/kicad/+bug/1832604
This commit is contained in:
Seth Hillbrand 2019-06-18 17:14:48 -07:00
parent 6444cfd345
commit b876309999
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
common/tool/tool_dispatcher.cpp
View File

@ -443,8 +443,10 @@ void TOOL_DISPATCHER::DispatchWxEvent( wxEvent& aEvent )
evt = TOOL_EVENT( TC_KEYBOARD, TA_KEY_PRESSED, key | mods );
}
bool handled = false;
if( evt )
m_toolMgr->ProcessEvent( *evt );
handled = m_toolMgr->ProcessEvent( *evt );
// pass the event to the GUI, it might still be interested in it
// Note wxEVT_CHAR_HOOK event is already skipped for special keys not used by KiCad
@ -467,7 +469,7 @@ void TOOL_DISPATCHER::DispatchWxEvent( wxEvent& aEvent )
// must be Skipped (sent to GUI).
// Otherwise accelerators and shortcuts in main menu or toolbars are not seen.
#ifndef __APPLE__
if( type == wxEVT_CHAR && !keyIsSpecial )
if( type == wxEVT_CHAR && !keyIsSpecial && !handled )
aEvent.Skip();
#endif
}