christoomey / vim-tmux-navigator

Seamless navigation between tmux panes and vim splits
MIT License
5.07k stars 319 forks source link

fix: workaround netrw mapping for ctrl-L #393

Closed nfischer closed 1 month ago

nfischer commented 1 month ago

This adds a workaround for the netrw keymapping conflict for ctrl-l by setting the g:Netrw_UserMaps list. To limit the impact to users, this will leave the list alone if the user has already set the variable and instead gives a warning to prompt the user about the issue.

This mapping workaround can be disabled by setting either g:tmux_navigator_no_mappings or g:tmux_navigator_disable_netrw_workaround.

This makes PR #392 obsolete.

Fixes #189, Fixes #251, Fixes #379

nfischer commented 1 month ago

Hey Chris, big fan of the plugin! It looks like this netrw issue has been around for a while and folks have figured out a few different kinds of workarounds. Let me know if you'd be open to making this workaround built into the vim-tmux-navigator plugin itself.

If it's too risky to work around this automatically, then I think PR #392 would be a really nice way to make people aware of how they can workaround the problem on their own.

christoomey commented 1 month ago

This is great! Thanks for detailed writeup and consideration. I think having this be built in makes sense I'll merge this one and comment on the other