smartparens icon indicating copy to clipboard operation
smartparens copied to clipboard

Ignore double backslashes (newlines) in Latex (#784)

Open rgemulla opened this issue 8 years ago • 4 comments

Fixes the roughest issues of #784; see the discussion there.

rgemulla avatar Oct 03 '17 15:10 rgemulla

Here is a summary of what this PR fixes and what's still open. I am not sure where to look for fixing the open issues, but I do think that this PR improves handling LaTeX files already substantially. From my point of view, it can be merged.

Now works correctly

Insertion and pair highlighting

()
\(\)
\\()
\\\(\)
\\\(\\\)

Mismatch highlighting:

(\)
\\(\)

Still does not work

Highlighting only shown when point on opening paren:

(\\)

Backspace deletion removes too much (should only remove "(", but removes "\(")

\\(

Prefixes not detected

\emph{abc}

Tags don't work at all (neither triggers nor highlighting):

\begin{quote}
...
\end{quote}

rgemulla avatar Nov 15 '17 23:11 rgemulla

In my point of view, this PR can be merged. There are still some problems open with LaTeX, but the PR nevertheless fixes many of the problems regarding backslashes.

rgemulla avatar May 31 '18 09:05 rgemulla

Not sure why the checks fail; it does not seem to be related to this PR though.

rgemulla avatar May 31 '18 09:05 rgemulla

Yea, tests are failing again, I think something in cask/evm broke.

I'll try to run this locally. Some additional tests for the new features would also be nice to add.

Fuco1 avatar May 31 '18 09:05 Fuco1