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

fish_user_keybindings #21

Closed
maxfl opened this Issue May 31, 2012 · 6 comments

Comments

Projects
None yet
4 participants
@maxfl
Contributor

maxfl commented May 31, 2012

I think it worth supporting fish_user_keybindings function, which should be loaded right after fish_default_keybindings to support user key bindings.

@adisbladis

This comment has been minimized.

Show comment
Hide comment
@adisbladis

adisbladis May 31, 2012

Contributor

Isn't this something that you should place in your config.fish instead?

Contributor

adisbladis commented May 31, 2012

Isn't this something that you should place in your config.fish instead?

@maxfl

This comment has been minimized.

Show comment
Hide comment
@maxfl

maxfl May 31, 2012

Contributor

Fish loads bindings after reading config.fish, and just before doing it, it resets existing bindings.

Contributor

maxfl commented May 31, 2012

Fish loads bindings after reading config.fish, and just before doing it, it resets existing bindings.

@maxfl maxfl closed this May 31, 2012

@maxfl maxfl reopened this May 31, 2012

@adisbladis

This comment has been minimized.

Show comment
Hide comment
@maxfl

This comment has been minimized.

Show comment
Hide comment
@maxfl

maxfl Jun 5, 2012

Contributor

Thank you very much. I'll check it as soon as it will merged.

Maxim
On Tue, 05 Jun 2012 21:56:04 +0400, adisbladis
reply@reply.github.com
wrote:

https://gitorious.org/~ridiculousfish/fish-shell/fishfish/merge_requests/5


Reply to this email directly or view it on GitHub:
https://github.com/ridiculousfish/fishfish/issues/21#issuecomment-6131720

Contributor

maxfl commented Jun 5, 2012

Thank you very much. I'll check it as soon as it will merged.

Maxim
On Tue, 05 Jun 2012 21:56:04 +0400, adisbladis
reply@reply.github.com
wrote:

https://gitorious.org/~ridiculousfish/fish-shell/fishfish/merge_requests/5


Reply to this email directly or view it on GitHub:
https://github.com/ridiculousfish/fishfish/issues/21#issuecomment-6131720

@ridiculousfish

This comment has been minimized.

Show comment
Hide comment
@ridiculousfish

ridiculousfish Jun 12, 2012

Member

Should have been merged with this commit:
ca61d0e..64afada fish_fish –> fish_fish

Member

ridiculousfish commented Jun 12, 2012

Should have been merged with this commit:
ca61d0e..64afada fish_fish –> fish_fish

@mpe

This comment has been minimized.

Show comment
Hide comment
@mpe

mpe Jun 18, 2012

Is this documented somewhere?

mpe commented Jun 18, 2012

Is this documented somewhere?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment