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

[Bug] Esc in environment editor with Vim mode not working #208

Closed
antonbasic opened this issue May 15, 2017 · 4 comments · Fixed by #1429
Closed

[Bug] Esc in environment editor with Vim mode not working #208

antonbasic opened this issue May 15, 2017 · 4 comments · Fixed by #1429
Labels
N-discussion Needs: Discussion

Comments

@antonbasic
Copy link

Overview

  • Insomnia Version: v5.0.20
  • Operating System: MacOS 10.12.4 (16E195)
  • Summary: When editing an environment using vim key map you cannot use esc to exit insert mode. Esc instead causes the settings modal to close.

How To Reproduce

  1. Set Text Editor Key Map to Vim
  2. Go to Manage environments
  3. In the editor, press i to enter insert mode.
  4. Try to exit insert mode by pressing esc
@gschier gschier changed the title Esc in environment editor with Vim mode not working [Bug] Esc in environment editor with Vim mode not working May 15, 2017
@gschier
Copy link
Contributor

gschier commented May 15, 2017

Hmm... this may be a tricky one to fix. The Vim mode is provided directly by Codemirror. All this probably needs is an e.stopPropagation() when the escape command fires but, without forking the Codemirror vim mode, might be hard to do.

@gschier gschier added the N-discussion Needs: Discussion label May 15, 2017
@gschier
Copy link
Contributor

gschier commented Sep 18, 2017

Closing as stale.

@gschier gschier closed this as completed Sep 18, 2017
@kitsu
Copy link

kitsu commented Feb 22, 2019

Please address this. All you need to do is disable ESC closing the environment modal (which already has two buttons for closing).

Edit: As a work-around C-c works like escape (C-] does not).

@gschier
Copy link
Contributor

gschier commented Mar 7, 2019

@kitsu if you want to submit a PR for this I'd be happy to take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
N-discussion Needs: Discussion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants