adobe/brackets — pull request #2421 by jasonsanjose (2012-12-20 20:44:36+00:00)
Title: Jasonsanjose/issue 2415

#2415

* Removes console.log for duplicate modifier key check
* Allow existing generic bindings to be overridden by platform-specific bindings
* Bonus: add node_modules to .gitignore
* Update unit tests

Note: There can still be some console spam on mac depending on the order of loading the modules for these commands: edit.findNext, edit.selectLine and navigate.gotoLine. This is by design.

Basically, if navigate.gotoLine is loaded last, the generic Ctrl-G tries to clobber the platform-specific Cmd-G for edit.findNext. I couldn't come up with an elegant way around this that violate our best practice for always defining generic key bindings.

```
"edit.selectLine": [
{
"key": "Ctrl-L"
},
{
"key": "Ctrl-L",
"platform": "mac"
}
],
"edit.findNext": [
{
"key": "F3"
},
{
"key": "Cmd-G",
"platform": "mac"
}
],
"navigate.gotoLine": [
{
"key": "Ctrl-G"
},
{
"key": "Cmd-L",
"platform": "mac"
}
],
```