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

Auto-delete auto-closed delimiters #1837

Closed
nekodjin opened this issue Mar 18, 2022 · 2 comments
Closed

Auto-delete auto-closed delimiters #1837

nekodjin opened this issue Mar 18, 2022 · 2 comments
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements R-duplicate Duplicated issue: please refer to the linked issue

Comments

@nekodjin
Copy link

From an implementation standpoint, the request is as follows: "Whenever a character is deleted, check if it forms an auto-closing delimiter pair with the antecedent character. If so, delete both."

Here's a concrete example: I type (. Helix detects that this is the first in an auto-closing delimiter pair, and therefore inserts the corresponding ) directly after my cursor. I hit backspace, which deletes the original ( character, but the automatically inserted ) character remains. This feature request requests that the automatically inserted ) should be deleted.

@nekodjin nekodjin added the C-enhancement Category: Improvements label Mar 18, 2022
@kirawi kirawi added the A-helix-term Area: Helix term improvements label Mar 18, 2022
@nekodjin
Copy link
Author

Revisiting this with an additional suggestion. When the first character of a delimiter pair is typed, if it would close a previously unclosed closing delimiter pair, it should not have the corresponding closing character inserted afterwards. For example: if my text looks like some | stuff), where | represents my cursor, and I type (, the corresponding ) should not be automatically inserted.

@kirawi
Copy link
Member

kirawi commented Jul 14, 2022

Duplicate of #1673

@kirawi kirawi marked this as a duplicate of #1673 Jul 14, 2022
@kirawi kirawi closed this as not planned Won't fix, can't repro, duplicate, stale Jul 14, 2022
@kirawi kirawi added the R-duplicate Duplicated issue: please refer to the linked issue label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements R-duplicate Duplicated issue: please refer to the linked issue
Projects
None yet
Development

No branches or pull requests

2 participants