Keyboard shortcuts disabled for alternate layouts

[oBFusCATed]

OK, I'll try to log the issue on your behalf...

[oBFusCATed]

Ticket posted https://trac.wxwidgets.org/ticket/18370

[hidefromkgb]

Heads up, I`ve got news!

This is where the error seems to happen:

Code (cpp) Select Expand

int  ScintillaWX::DoKeyDown(const wxKeyEvent& evt, bool* consumed)
{
    int key = evt.GetKeyCode();
    if (key == WXK_NONE) {
        // This is a Unicode character not representable in Latin-1 or some key
        // without key code at all (e.g. dead key or VK_PROCESSKEY under MSW).
        if ( consumed )
            *consumed = false;
        return 0;
    }

    ...

WxWidgets Scintilla implementation processes shortcuts not by the keyboard scancode as it needs to be, but by the actual character typed!
Do you guys know how to retrieve scancodes from a wxKeyEvent ?

[hidefromkgb]

Update: found it! It`s called wxKeyEvent::GetRawKeyCode(), but its results are platform dependent. Okay that`s fine, I`ve already collected the scancode tables for Windows, Linux and MacOS =)

[oBFusCATed]

Please post any of these in the wx's trac...
Here they would get lost...

[hidefromkgb]

Can I just post a .DIFF patch to wxTRAC?
Something tells me I`m capable of fixing that problem all by myself.

[oBFusCATed]

Yes, you can.

[hidefromkgb]

Done! PR #1305.

[hidefromkgb]

...Guys, would it be too much if I asked you to apply that patch to C::B ahead of the WXW trunk?
Granted, I can just take the source and do that myself, but it`d be way cool if that bugfix made it to C::B, for I doubt I`m the only one experiencing that problem.

[oBFusCATed]

I'm hesitant to do it, because syncing the source is already hard and applying this patch it will make it harder.
I guess New Pagodi could help test the patch?

[hidefromkgb]

Fix applied!
https://github.com/wxWidgets/wxWidgets/commit/bc1295fbdc2849013333c7107c5abfc4a04ac6b0

[oBFusCATed]

I'll pick the change next time I update wxscintilla...

[oBFusCATed]

In svn...