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

Wrong command name in mathtools: DeclarePairedDelimiters should be DeclarePairedDelimiter #2758

Closed
robly18 opened this issue Aug 10, 2021 · 2 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed v3 v3.2
Milestone

Comments

@robly18
Copy link

robly18 commented Aug 10, 2021

The mathtools package in LaTeX has a command \DeclarePairedDelimiter. Mathjax reproduces it, but the name has an extra "s" at the end: \DeclarePairedDelimiters, as do the variations. For example, \DeclarePairedDelimitersX should instead be \DeclarePairedDelimiterX.

I have found no reason for this change, so as far as I know it is a typo. If it is on purpose, then perhaps a warning should be placed on the mathtools page.

@zorkow
Copy link
Member

zorkow commented Aug 17, 2021

This is probably a typo. Even if \DeclarePairedDelimiters and \DeclarePairedDelimitersX were to remain, the singular version should be added.

@zorkow zorkow added the Accepted Issue has been reproduced by MathJax team label Aug 17, 2021
@dpvc dpvc added the v3 label Sep 25, 2021
@dpvc dpvc self-assigned this Sep 25, 2021
@BlackHC
Copy link

BlackHC commented Oct 21, 2021

And same for DeclarePairedDelimitersXPP -> DeclarePairedDelimiterXPP.

I just ran into the same issue and took me a few hours of debugging to figure out what is going on 🥲

Thanks so much!

dpvc added a commit to mathjax/MathJax-src that referenced this issue Jan 18, 2022
@dpvc dpvc added this to the 3.2.1 milestone Jan 31, 2022
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 22, 2022
Fix typo in \DeclarePairedDelimiter*, and substitute arguments in pre and post sections. (mathjax/MathJax#2816, mathjax/MathJax#2758)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Feb 22, 2022
@dpvc dpvc removed their assignment Jun 1, 2022
@dpvc dpvc added Fixed v3.2 and removed Merged Merged into develop branch labels Jun 1, 2022
@dpvc dpvc closed this as completed Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed v3 v3.2
Projects
None yet
Development

No branches or pull requests

4 participants