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

Prevent "Press ENTER or type command to continue" #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Prevent "Press ENTER or type command to continue" #24

wants to merge 1 commit into from

Conversation

Cade-Forester
Copy link

Do not show "Press ENTER or type command to continue" message after type ".". Prepend "silent!"

Do not show "Press ENTER or type command to continue" message after type ".". Prepend "silent!"
@tpope
Copy link
Owner

tpope commented Mar 30, 2014

How do I reproduce? :silent! hides all errors so I don't want to introduce it blindly.

@Cade-Forester
Copy link
Author

Steps to reproduce.

  • Create file /tmp/test.vim:
set runtimepath+=~/.vim/bundle/vim-repeat
set columns=40
function! Test____________________________________ ()
   normal! dd
   silent! call repeat#set( ":call Test____________________________________()\n", v:count )
endfunction
nnoremap <silent> dd :call Test____________________________________()<cr>

Function has a long name.

  • Start vim:
vim -u NONE -U NONE -i NONE --noplugin -n -N
  • Type:
source /tmp/test.vim
  • Open some file
:edit file
  • Type
dd

and

.

Result: message
"Press ENTER or type command to continue"

Sorry, i just test that my patch does not solve problem.
Need replace "feedkeys(cmd)" on ":execute cmd", but this break compatibility.

@tpope
Copy link
Owner

tpope commented Mar 30, 2014

You should be able to work around this by adding a <silent> <Plug>Test map in the middle to actually :call Test, and then feed that in as "\<Plug>Test".

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