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

Latex fixes #1151

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Latex fixes #1151

wants to merge 5 commits into from

Conversation

dschrempf
Copy link

Hi! This fixes #1100; well, it disables some features of smartparens for latex modes. The PR also includes some other nits and changes. Let me know if I should exclude those.

Finally, this change is a breaking change for people using

(sp-local-pair modes "``" nil ...))) ;; or
(sp-local-pair modes "`" nil ...)))

For example, this affects Doom Emacs, which I am using. The corresponding patch for Doom Emacs is attached.
s.patch

@Fuco1
Copy link
Owner

Fuco1 commented Mar 16, 2023

Hmm, to be fair though, the "default" behaviour is fine until you use those special language quotes, right? I wonder if we shouldn't keep it working e.g. in english.

@dschrempf
Copy link
Author

For me, smartparens is completely broken in LaTeX, also in English. I am not using German in my documents. It could be a Doom Emacs issue. I removed all the relevant changes (see the patch), but who knows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

smartparens breaks LaTeX quotation marks mechanism for language styles
2 participants