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

Canceling auto−complete popup by Escape trigger mode change to normal-mode #339

Closed
t9md opened this issue Aug 28, 2016 · 7 comments
Closed

Comments

@t9md
Copy link
Owner

t9md commented Aug 28, 2016

I noticed this via this tweet

autocomplete-plus also using escape, and its bound to auto-complete-plus:cancel
https://github.com/atom/autocomplete-plus/blob/master/keymaps/autocomplete-plus.cson#L5

Because atom command is propagated, escape to cancel autocomplete-plus's popup also trigger vim-mode-plus:activate-normal-mode.
I this is not normal user expect, what normal user expect is escape only close popup and remain in insert-mode.
If user really want to escape insert-mode, another escape should be typed.
Current behavior taking the choice by user. Not good.

@t9md t9md changed the title Canceling auto−complete popup by Escape trigger mode change to normal-mode 💣 Canceling auto−complete popup by Escape trigger mode change to normal-mode Aug 28, 2016
@t9md t9md changed the title 💣 Canceling auto−complete popup by Escape trigger mode change to normal-mode Canceling auto−complete popup by Escape trigger mode change to normal-mode Aug 28, 2016
t9md added a commit that referenced this issue Aug 28, 2016
By this change escape is no longer trigger activate-normal-mode when
autocomplete-active is set in classList
@t9md t9md closed this as completed Aug 28, 2016
@lukebarton
Copy link

This issue brought me here, because it was resolved - it's now super annoying to hit escape twice, just because the autocomplete prompt happened to be showing.

Can we have an option?

@t9md
Copy link
Owner Author

t9md commented Sep 3, 2016

Oh understand. Sorry.
I don't want to provide option.
What Im thinking is to revert to previous behavior.
Original issue have simple solution to set explicit keymap to close autocomplete popup.
Thanks for notifying this. Will revert soon.

@lukebarton
Copy link

lukebarton commented Sep 3, 2016

In the meantime:

'atom-text-editor.vim-mode-plus:not(.normal-mode)':
  'escape': 'vim-mode-plus:activate-normal-mode'

I actually thought my escape key was broken because I didn't understand why I wasn't exiting insert mode 😂

@averrin
Copy link

averrin commented Sep 5, 2016

Yeah, I want close autocomplete AND escape insert mode. Can you release this revert?

@t9md
Copy link
Owner Author

t9md commented Sep 5, 2016

released patch as v0.54.1

@averrin
Copy link

averrin commented Sep 5, 2016

Thanks! 👍

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

No branches or pull requests

3 participants