Fixed Live Snippet Cursor Steps out of Parentheses

[josiahnross]

Closes #384

I don't know what the purpose of the removed lines was. It seems like they were there to compensate for the additional characters added by the replacement however I found that vscode handled the cursor placement fine on its own. So instead of fixing a problem, these lines were creating an issue by moving the cursor forward after the snippet has been inserted. This often led to the cursor being moved outside of parentheses and even the math mode entirely which was quite annoying to deal with

[leoleoasd]

Can you make sure that this works with other snippets? As I'm not the one who wrote this, it would take me a while to figure out what this does either.

[josiahnross]

It works with all snippets I have tested it on which include the following (these work in a variety of math modes (inline, align environment, etc):

• auto subscript
• whitespace before operators
• whitespace after operators
• ...
• implies
• fractions
• not equal
• aligned equal
• approx
• greater than or equal to
• times
• cdot
• pi
• overline
• hat
• partial derivative
• sqrt
• exists
• definite integral
• plus minus

[grepgrok]

From what I can tell from the API, editBuilder.replace above simply deletes whatever text is in the range and then inserts the string. This doesn't seem to affect the positioning of the cursor; that is to say, the cursor maintains its position relative to the rest of the text. Since the cursor isn't really moving, there should be no need to manually move it. So, these lines should be removed.