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

\left( inserts incorrect paired brace #3493

Open
jojo2357 opened this issue Mar 17, 2024 · 3 comments · Fixed by #3562
Open

\left( inserts incorrect paired brace #3493

jojo2357 opened this issue Mar 17, 2024 · 3 comments · Fixed by #3562
Labels
enhancement New feature or (non bug related) change to the program.

Comments

@jojo2357
Copy link
Contributor

In a math environment, doing \left( should, imo, automatically insert \right) rather than )

IE \left<carret> -> type ( -> \left(<caret>)
expected \left<carret> -> type ( -> \left(<caret>\right)

@jojo2357 jojo2357 added enhancement New feature or (non bug related) change to the program. untriaged Issue type still needs to be triaged or verified. labels Mar 17, 2024
@PHPirates
Copy link
Collaborator

I agree but on the condition that we can ensure the \right) is only inserted if the ) is inserted, so if there is no matching ) already present, for example \left<caret> (...) \right)

@PHPirates PHPirates removed the untriaged Issue type still needs to be triaged or verified. label Mar 20, 2024
@jojo2357 jojo2357 changed the title \left( does not autocomplete with \right) \left( inserts incorrect paired brace May 4, 2024
@slideclimb slideclimb self-assigned this May 26, 2024
slideclimb added a commit that referenced this issue May 26, 2024
@PHPirates PHPirates added this to the Next milestone May 27, 2024
@jojo2357
Copy link
Contributor Author

That did not fix my issue.

My test, characterized by:

$\left<caret> $

typing (

expected:

$\left(<caret>\right) $

actual:

$\left(<caret>) $

the unit test described above can be characterized below:

fun testCorrectPairedBrace() {
    myFixture.configureByText(LatexFileType, """$\left<caret>$""")
    myFixture.type('(')
    myFixture.checkResult("""$\left(<caret>\right)$""")
}

Is not fixed by c20606a. #3562 did fix a fun bug, but not this issue.

Please reopen/

@PHPirates
Copy link
Collaborator

Thanks, closed this by mistake

@PHPirates PHPirates reopened this May 29, 2024
@PHPirates PHPirates removed this from the 0.9.6 milestone Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or (non bug related) change to the program.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants