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

Update keybindings.go #162

Merged
merged 2 commits into from
Sep 3, 2019
Merged

Update keybindings.go #162

merged 2 commits into from
Sep 3, 2019

Conversation

Tuurlijk
Copy link
Contributor

@Tuurlijk Tuurlijk commented Sep 3, 2019

Remove duplicate keybinding: volumes - c
Bind F10 to gui.quit

Remove duplicate keybinding: volumes - c
Bind F10 to gui.quit
Repository owner deleted a comment from codecov-io Sep 3, 2019
Repository owner deleted a comment from codecov-io Sep 3, 2019
Repository owner deleted a comment from codecov-io Sep 3, 2019
@codecov-io

This comment has been minimized.

@dawidd6
Copy link
Collaborator

dawidd6 commented Sep 3, 2019

Nice, but why F10 tho?

@Tuurlijk
Copy link
Contributor Author

Tuurlijk commented Sep 3, 2019

Muscle memory from the Norton Commander / Midnight Commander days

@glvr182
Copy link

glvr182 commented Sep 3, 2019

I would say that this can be done by the users themselfs when #117 is merged. No need to make this into a hardcoded config

@Tuurlijk
Copy link
Contributor Author

Tuurlijk commented Sep 3, 2019

Ok, but remove the duplicate volume - c keybinding.

@dawidd6
Copy link
Collaborator

dawidd6 commented Sep 3, 2019

Can you make another commit removing this new binding? @Tuurlijk

Will merge this afterwards.

@Tuurlijk Tuurlijk closed this Sep 3, 2019
@Tuurlijk Tuurlijk reopened this Sep 3, 2019
@glvr182
Copy link

glvr182 commented Sep 3, 2019

Ok, but remove the duplicate volume - c keybinding.

Yea I removed that in the PR.

@dawidd6 dawidd6 merged commit ea18c1b into jesseduffield:master Sep 3, 2019
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.

4 participants