mini.pick: choose_in_split mapping not working as expected

[floydj]

Contributing guidelines

• I have read CODE_OF_CONDUCT.md

Module(s)

mini.pick

Question

I customize the keybinding for choosing a file in a horizontal split with the following in init.lua:

require('mini.pick').setup({
  mappings = {
    choose_in_split = '<C-i>'
  }
})

Sometimes this works as I expect, but more often, it opens the file in the preview window, not a new split buffer. If I keep pressing <C-i, it'll toggle between the file I picked, and showing the path location of the file.

It always works correctly if I erase the above and just use the standard <C-s> for opening in a split, but that's a deal-breaker for me, as it's already used by a customization I have in macOS.

Am I missing something with the configuration? If not, can you please suggest how I can debug further? I've looked over the documentation, but can't see anything I'm doing wrong. I've also ensured that I haven't (purposely at least) setup the <C-i> keybind elsewhere.

Other mini modules I'm using, in addition to mini.deps:

require('mini.diff').setup()
require('mini.extra').setup()
require('mini.files').setup()
require('mini.icons').setup()
require('mini.indentscope').setup()
require('mini.sessions').setup()
require('mini.snippets').setup()
require('mini.starter').setup()
require('mini.surround').setup()
require('mini.trailspace').setup()
require('mini.visits').setup()

[echasnovski]

The <C-i> and <Tab> is usually interpreted by the terminal emulator as the same key (see this, for example). So in this case this setup is the same as choose_in_split = '<Tab>', which makes it conflict with the default toggle_preview action (which is also <Tab>).

The reason this sometimes works is because actions are not guaranteed to be preferred in any order. Sometimes <Tab> acts as choose_in_split and sometimes as toggle_preview.

The solution is to make toggle_preview be different from choose_in_split, i.e. use different key for either choose_in_split or toggle_preview.

[floydj]

Makes sense. Thanks!