Add matchbraceleft option
#3432
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add `matchbraceleft` option to allow disabling the default behavior matching not just the brace under cursor but also the brace to the left of it (which is arguably convenient, but also ambiguous and non-intuitive). With `matchbraceleft` disabled, micro will only match the brace character that is precisely under the cursor, and also when jumping to the matching brace, will always move cursor precisely to the matching brace character, not to the character next to it. Nota bene: historical journey: - There was already a `matchbraceleft` option introduced in commit ea6a87d, when this feature (matching brace to the left) was introduced first time. That time it was matching _only_ the brace to the left, _instead_ of the brace under the cursor, and was disabled by default. - Later this feature was removed during the big refactoring of micro. - Then this feature was reintroduced again in commit d1e713c, in its present form (i.e. combined brace matching both under the cursor and to the left, simulating I-beam cursor behavior), and it was introduced unconditionally, without an option to disable it. - Since then, multiple users complained about this feature and asked for an option to disable it, so now we are reintroducing it as an option again (this time enabled by default though).
This was referenced Aug 18, 2024
JoeKar
approved these changes
Aug 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add
matchbraceleftoption to allow disabling the default behavior matching not just the brace under cursor but also the brace to the left of it (which is arguably convenient, but also ambiguous and non-intuitive). Withmatchbraceleftdisabled, micro will only match the brace character that is precisely under the cursor, and also when jumping to the matching brace, will always move cursor precisely to the matching brace character, not to the character next to it.Nota bene: historical journey:
There was already a
matchbraceleftoption introduced in commit ea6a87d, when this feature (matching brace to the left) was introduced first time. That time it was matching only the brace to the left, instead of the brace under the cursor, and was disabled by default.Later this feature was removed during the big refactoring of micro.
Then this feature was reintroduced again in commit d1e713c, in its present form (i.e. combined brace matching both under the cursor and to the left, simulating I-beam cursor behavior), and it was introduced unconditionally, without an option to disable it.
Since then, multiple users complained about this feature and asked for an option to disable it (e.g. Jumping to matching brace not as expected #3419 (comment)), so now we are reintroducing it as an option again (this time enabled by default though).
Fixes #3419