jethrokuan / fzf

Ef-🐟-ient fish keybindings for fzf
MIT License
861 stars 66 forks source link

Don't bind tab if tab has bindings already #169

Closed jorgebucaran closed 3 years ago

jorgebucaran commented 3 years ago

Tab support is tricky. I don't propose we remove tab binding support outright, but we could improve our situation a bit.

By merging this PR, fzf will no longer bind tab if bindings for tab are already present. I considered wrapping existing tab bindings instead, but because fzf takes partial control of the screen it's difficult to imagine fzf tab bindings being compatible with any other plugin.

This would help with simple cases like https://github.com/jethrokuan/fzf/issues/167.