Skip to content

Added config and setup(), handler(), Add which-key support, allow items= to be a function, more README, add some types, automatic filetype items #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

Kamilcuk
Copy link
Contributor

@Kamilcuk Kamilcuk commented Apr 13, 2025

Hi:

  • Create require("menu").config require("menu").setup() for configuration
  • Added require("menu").handler() that does the same as from README.
  • I created which-key menu to draw a full recursive menu starting from mapleader, which I think will be the most useful for users.
  • items= can now be a function, like items = function() require("menus.which-key") end. This is done to lazy-load the items when they are opened. There is no need to load every submenu from the start.
  • Added menus/fs/ directory that has filetype specific menus and added support in handler() to detect them.
  • Added stylua.toml. I aimed for style closest to the existing formatting style.

Example which-key example of 4 menus:

image

@siduck
Copy link
Member

siduck commented Apr 14, 2025

this is getting complex, i wanted default file to just be an example

@Strongsloth
Copy link

would be cool, but is it good performance wise?

@Kamilcuk
Copy link
Contributor Author

hi @Strongsloth The which-key menu opens instantly for me in astronvim if that's what you are asking.

@Strongsloth
Copy link

yeah

@siduck
Copy link
Member

siduck commented Apr 22, 2025

whichkey doesnt open instantly for the first time in nvchad because

NvChad/NvChad#3036

@Strongsloth
Copy link

oh so that's why when I press the <Leader> then a prefix such as db or dd it doesn't instantly show whickkey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants