nixvim

nixvim copied to clipboard

First time opening a PR here, so let me know if I've missed some etiquette.

Summary

• New option tmuxNavigatorNoMappings maps to global tmux_navigator_no_mappings
• New attrset keymaps maps keymaps to the various :TmuxNavigate* commands
  • Set of either enum or mapConfigOptions submodule (with some tweaks)

Future refactoring

I don't see why the various options should include the tmuxNavigator prefix; that is surely implied by being in the tmux-navigator plugin...

I also think that the null or 1 options could be refactored as bool options, to make more sense in nix...

I can open this as a separate issue/PR if it warrants discussion, or I could lump it into this PR if that's easier to review?

I don't see why the various options should include the tmuxNavigator prefix; that is surely implied by being in the tmux-navigator plugin...

I also think that the null or 1 options could be refactored as bool options, to make more sense in nix...

I can open this as a separate issue/PR if it warrants discussion, or I could lump it into this PR if that's easier to review?

IMO this PR should be reworked after the plugin is ported to mkVimPlugin. I've opened #1293 to do that.

Is this PR still relevant ? If so, it is necessary to rebase it on main as it has been a while ^^

cc @MattSturgeon

Unless I have some time to kill sooner, I'll take a look when I get back to the UK.

It'll probably need a rewrite due to #1293.