Skip to content
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

Can't use double-quote in a mapping #1212

Closed
10 of 15 tasks
flwyd opened this issue Jan 30, 2021 · 0 comments · Fixed by #1213
Closed
10 of 15 tasks

Can't use double-quote in a mapping #1212

flwyd opened this issue Jan 30, 2021 · 0 comments · Fixed by #1213
Labels

Comments

@flwyd
Copy link

flwyd commented Jan 30, 2021

Self-Diagnosis

  • I have searched the issues for an answer to my question.
  • I have reviewed the NERDTree documentation. :h NERDTree
  • I have reviewed the Wiki.
  • I have searched the web for an answer to my question.

Environment (for bug reports)

  • Operating System: MacOS
  • Vim/Neovim version :echo v:version: 802
  • NERDTree version, found on 1st line in NERDTree quickhelp ?: 6.10.0
  • vimrc settings
    • NERDTree variables
    function! QuoteMapping() abort
      echo "Hello, world!"
    endfunction
    call NERDTreeAddKeyMap({'key': ',"', 'callback': 'QuoteMapping', 'quickhelpText': 'say hello'})
    • Other NERDTree-dependent Plugins
      • jistr/vim-nerdtree-tabs
      • ryanoasis/vim-devicons
      • tiagofumo/vim-nerdtree-syntax-highlight
      • Xuyuanp/nerdtree-git-plugin
      • Others (specify):
    • I've verified the issue occurs with only NERDTree installed.

Steps to Reproduce the Issue

  1. Declare a NERDTree key mapping that involves a double-quote (")
  2. Open NERDTree
  3. Use the mapping

Current Result (Include screenshots where appropriate.)

E116: Invalid arguments for function nerdtree#ui_glue#invokeKeyMap

Expected Result

Mappings can include any ASCII character.

This can probably be addressed like #948. I don't know if any other characters also need to be escaped.

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

Successfully merging a pull request may close this issue.

1 participant