[mini.files] Multiple keys per mapping

[louwers]

I'd like to be able to define multiple keys per mapping.

Would a PR for this feature be accepted?

require("mini.files").setup({
	mappings = {
		go_in_plus = { '<CR>', 'L' },
	},
})

[echasnovski]

This is already possible and was discussed in #1629, #1309, #879, #534, and #409.

[louwers]

Ah thanks, sorry I missed those.

Since multiple people independently suggested this API, maybe it could be added? Would be a lot simpler than copy pasting an internal function + adding an autocmd.

[echasnovski]

Would be a lot simpler than copy pasting an internal function + adding an autocmd.

No internal function is needed for go_in_plus action (and for most other built-in actions, for that matter). MiniFiles.go_in() supports close_on_file option which can be set to true to have go_in_plus behavior.

The autocommand approach is already present and a lot more flexible than making configuration code more complex. So although the "multiple people suggested" is a valid reasoning (if not taking prior searching of issues into account), I don't think I like the idea of allowing multiple keys per action in the MiniFiles.config itself.