only one shortcut per command

[to266]

It seems that currently the shortcuts only support one per command (hence #320 J/K in the notebook are not easily possible)

this commit would ideally add the shortcuts, but since they are higher than the original (arrow) ones, the arrow shortcuts get ignored.

Looks like the commands are a hash table... Alternatively I'm not adding things correctly (and should have added directly to the sequence somehow?)..

---

edit:
relevant part of the phosphide
https://github.com/phosphorjs/phosphide/blob/master/src/services/shortcutregistry.ts#L50

[blink1073]

Hi @to266, this behavior is fixed in the new consolidated phosphor repo, which we will be switching over to in the next week or so.

[jasongrout]

@blink1073 - done?

[blink1073]

Yep, fixed by #591.