Add support for grouped keys#1063
Conversation
|
Anything blocking this from getting merged? |
wbthomason
left a comment
There was a problem hiding this comment.
Thanks! Sorry for not seeing this before now. I think this is mostly good to go, but the updated language in the docs is technically incorrect, so we need to fix that before merging.
I'll have limited internet for the next week, but please @ me if you get to this and I haven't responded by Dec 21.
| #### Keybindings | ||
|
|
||
| Plugins may be lazy-loaded on the use of keybindings/maps. Individual keybindings are specified either as a string (in which case they are treated as normal mode maps) or a table in the format `{mode, map}`. | ||
| Plugins may be lazy-loaded on the use of keybindings/maps. Individual keybindings are specified either as a string (in which case they are treated as normal mode maps) or a table in one of the following formats: |
There was a problem hiding this comment.
Here and in the doc/ file, we need to change the wording a bit. It currently only talks about individual keybindings, but we want to say that you can specify the whole set of keybindings in one of these two table formats, not individual keybindings.
There was a problem hiding this comment.
Sorry for the late reply! Do you think they're clearer now?
|
Hey @musjj! thanks for your great PR, which I really need. Therefore, when will we fix the docs?🤗 |
If you have a lot of keys that you want to use for lazy loading, your config can become a bit hairy:
With this PR, you can now instead group your keys like this: