Skip to content

fix: recommended setup's lazy-loading breaks default keymaps - #23

Merged
2KAbhishek merged 2 commits into
2KAbhishek:mainfrom
nickjvandyke:fix-lazy
Jan 20, 2026
Merged

fix: recommended setup's lazy-loading breaks default keymaps#23
2KAbhishek merged 2 commits into
2KAbhishek:mainfrom
nickjvandyke:fix-lazy

Conversation

@nickjvandyke

@nickjvandyke nickjvandyke commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

Thanks for this plugin, it has saved me many a trip to the nerdy font website!

What does the PR do? (Required)

cmd = 'Nerdy' in the recommended lazy.nvim setup means the plugin is not loaded until that command. Thus the internal default keymaps are not created and the user must manually run :Nerdy to load and use the plugin.

I chose to solve this by moving the internal keymaps to lazy.nvim's keys array, so the plugin loads on these keymaps. This fixes the lazy-loading issue and adheres to best practices 馃檪

But you could also just remove the cmd key from the recommended lazy.nvim config.

Checklist (Required)

  • I have tested the changes on my local machine
  • I have added relevant documentation and tests for the changes
  • I have followed the style guidelines of this project

Evidence (Required)

Without this PR: Enters insert mode because the keys aren't mapped.

Screen.Recording.2026-01-16.at.8.07.23.AM.mov

With this PR: Opens select as expected.

Screen.Recording.2026-01-16.at.8.06.33.AM.mov

@nickjvandyke nickjvandyke changed the title fix: lazy-loading breaks default keymaps fix: recommended setup's lazy-loading breaks default keymaps Jan 16, 2026
@2KAbhishek

Copy link
Copy Markdown
Owner

@NickvanDyke thanks for raising the PR, I will make a few minor tweaks after merge :D

@2KAbhishek
2KAbhishek merged commit 066d3b7 into 2KAbhishek:main Jan 20, 2026
3 checks passed
@nickjvandyke
nickjvandyke deleted the fix-lazy branch January 20, 2026 22:09
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.

2 participants