Skip to content

fix: pass function name when defining mappings#388

Merged
lervag merged 1 commit intolervag:masterfrom
rvmtz:master
Nov 3, 2024
Merged

fix: pass function name when defining mappings#388
lervag merged 1 commit intolervag:masterfrom
rvmtz:master

Conversation

@rvmtz
Copy link
Copy Markdown
Contributor

@rvmtz rvmtz commented Nov 3, 2024

Ensures g:wiki_select_method is evaluated during mapping definition,
allowing bang_wrapper() to receive the function name instead of a variable name.

Ensures `g:wiki_select_method` is evaluated during mapping definition,
allowing `bang_wrapper()` to receive the function name instead of a variable name.
@rvmtz
Copy link
Copy Markdown
Contributor Author

rvmtz commented Nov 3, 2024

@lervag , thanks for sharing your plugin. I use it on a daily basis. 😅
Since the last update, the :WikiPages command stopped working, I was wondering why and ended up submitting this patch.
Hope you find it useful. 🙂

@lervag lervag merged commit 982a54c into lervag:master Nov 3, 2024
@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 3, 2024

@lervag , thanks for sharing your plugin. I use it on a daily basis. 😅

Thanks, very happy to hear that!

Since the last update, the :WikiPages command stopped working, I was wondering why and ended up submitting this patch. Hope you find it useful. 🙂

And sorry about that; I forgot to test these. Thanks for noticing the error and pushing a fix for it! Very well done!

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