Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add two new key binding a fix another one. #7

Merged
merged 1 commit into from
Feb 1, 2013
Merged

Add two new key binding a fix another one. #7

merged 1 commit into from
Feb 1, 2013

Conversation

fcuny
Copy link

@fcuny fcuny commented Feb 1, 2013

Add two new bindings: one for help ('?') and another one to delete a view ('d'). The last one is in addition to the 'delete' key, since some browser bind this to "previous page in history".

Also, the correct key code (at least on Mac's QWERTY keyboard) for the letter 'h' is 72.

Add two new bindings: one for help ('?') and another one to delete a view ('d').  The last one is in addition to the 'delete' key, since some browser bind this to "previous page in history".

Also, the correct key code (at least on Mac's QWERTY keyboard) for the letter 'h' is 72.
@fcuny
Copy link
Author

fcuny commented Feb 1, 2013

If there's a good reason to have vertical split bind to 87, let me know and I'll fix my patch.

@aphyr
Copy link
Collaborator

aphyr commented Feb 1, 2013

Ah, you're right, I was using w for hsplit. It probably should be h. :)

I don't want to change delete's binding, though. 46 is delete, 8 is backspace. I'll merge your patch, but change that binding back.

aphyr added a commit that referenced this pull request Feb 1, 2013
Add two new key binding a fix another one.
@aphyr aphyr merged commit 4d03c0d into riemann:master Feb 1, 2013
@aphyr
Copy link
Collaborator

aphyr commented Feb 1, 2013

Thanks for adding the help window, too! Added you to the contributors file. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants