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

Add option to disable auto-pair of quotes, parens, etc. #117

Closed
joewiz opened this issue Oct 26, 2015 · 1 comment · Fixed by #475
Closed

Add option to disable auto-pair of quotes, parens, etc. #117

joewiz opened this issue Oct 26, 2015 · 1 comment · Fixed by #475

Comments

@joewiz
Copy link
Member

joewiz commented Oct 26, 2015

I've never gotten used to eXide's auto-pair function. This is where you type a single " and get two - "" - with your cursor placed between them. The same behavior affects ', (, and {. Often, when I am trying to insert a single instance of these characters, the editor inserts two. Then I delete one, but the editor deletes both. This can be maddening and leads to many extraneous key presses. I imagine some developers are used to this behavior from other code editors, but for me - and I'd bet - many new users, typing one thing and having something else happen is disconcerting and leads to wasted keypresses.

Would it be possible to add a preference disabling this?

Caveat: I would hope that this preference wouldn't disable the autocompletion of elements, e.g., <x>as soon as I type </immediately becomes <x>as soon as I type </x>. This generally works well - not perfect, but on balance is a time saver. Also valuable is the snippet behavior, where I can type a code like html, tap control-space, and html is replaced by some XQuery code.

@marmoure
Copy link
Contributor

@joewiz This is pretty much the standard behaviour in every IDE at this point trying to do a double quote will always produce "" with the cursor in between ,to minimize clicks you can click on Del which should delete the back pair.
If you still think this is a problem we can try to take out keyboard control from ACE and try to manipulate it before passing it to the editor again.

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

Successfully merging a pull request may close this issue.

2 participants