handleKeys doesn't work in Ver 0.10.3

[michaelHL]

BUG REPORT

Environment:

• VSCode Version: 1.18.0
• VsCodeVim Version: 0.10.3
• OS: Windows 10 (1607) x64

What happened:

<C-n> give no response after updating to v0.10.3.

What did you expect to happen:

Open a new untitled file.

---

I have already set "<C-n>": false in "vim.handleKeys", and my settings of VsCodeVim is as follows:

settings.json

"vim.useSystemClipboard": false,
"vim.hlsearch": true,
"vim.leader": " ",
"vim.easymotion": true,
"vim.easymotionMarkerFontFamily": "Microsoft Yahei Mono",
"vim.easymotionMarkerFontSize": "18",
"vim.easymotionMarkerHeight": 17,
"vim.easymotionMarkerWidthPerChar": 11,
"vim.easymotionMarkerBackgroundColor": "#000000",
"vim.easymotionMarkerForegroundColorOneChar": "#ffffff",
"vim.easymotionMarkerForegroundColorTwoChar": "#ffffff",
"vim.easymotionMarkerYOffset": 13,
"vim.statusBarColorControl": true,
"vim.otherModesKeyBindingsNonRecursive": [{
        "before": ["leader", "w"],
        "after": [],
        "commands": [{
            "command": "workbench.action.files.save",
            "args": []
        }]
    },
    {
        "before": ["leader", "s"],
        "after": ["leader", "leader", "s"]
    },
    {
        "before": ["leader", "p"],
        "after": [],
        "commands": [{
            "command": "workbench.action.closeEditorsInGroup",
            "args": []
        }]
    },
    {
        "before": ["leader", "q"],
        "after": [],
        "commands": [{
            "command": "workbench.action.closeActiveEditor",
            "args": []
        }]
    },
    {
        "before": ["leader", "l"],
        "after": [],
        "commands": [{
            "command": "editor.action.trimTrailingWhitespace",
            "args": []
        }]
    },
    {
        "before": ["U"],
        "after": ["<C-r>"]
    },
    {
        "before": ["="],
        "after": []
    }
],
"vim.handleKeys": {
    "<C-s>": false,
    "<C-f>": false,
    "<C-j>": false,
    "<C-c>": false,
    "<C-n>": false,
    "<C-k>": false
},

[jpoon]

Dammit. This is my doing in the last release: https://github.com/VSCodeVim/Vim/blame/master/src/configuration/configuration.ts#L80.

I reversed the logic. The check should be if is this.handleKeys[bracketedKey]===false then we should set useKey=false.

I'm about to head out -- won't be able to get to this until the EOD. Feel free to submit a PR in the meantime. Sorry :(

[michaelHL]

Back to normal again! 😄
Thx a lot!!!