package-info.nvim icon indicating copy to clipboard operation
package-info.nvim copied to clipboard

[FEATURE REQUEST] Update Readme keymap config examples with description

Open dzintars opened this issue 3 years ago • 1 comments

Issues

  • [X] I have checked existing issues and there are no existing ones with the same request.

Feature description

I would suggest to update key binding examples to:

vim.api.nvim_set_keymap(
    "n",
    "<leader>ns",
    "<cmd>lua require('package-info').show()<cr>",
    { silent = true, noremap = true, desc = "package-info | Show" }
)

Note the , desc = "package-info | Show" addition.

While this is "opinionated" thing to do, I do believe that newer Nvim users will benefit of cleaner :map n or :Telescope keymaps or which-key key descriptions. This enables easier duplicate keybinding finding and seeing "what is what and from where does it comes".

Also, examples could be improved with vim.keymap.set()

Help

Yes

Implementation help

No response

dzintars avatar Dec 10 '22 21:12 dzintars

Makes sense. Feel free to open a PR.

vuki656 avatar Dec 20 '22 20:12 vuki656