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

Bug in DeclarePairedDelimitersXPP #2816

Closed
xiaoniu-578fa6bff964d005 opened this issue Jan 6, 2022 · 1 comment
Closed

Bug in DeclarePairedDelimitersXPP #2816

xiaoniu-578fa6bff964d005 opened this issue Jan 6, 2022 · 1 comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed v3 v3.2
Milestone

Comments

@xiaoniu-578fa6bff964d005
Copy link

xiaoniu-578fa6bff964d005 commented Jan 6, 2022

Issue Summary

If DeclarePairedDelimitersXPP in MathJax should have same function as DeclarePairedDelimiterXPP in LaTeX, then it should substitute pre and post part.

Steps to Reproduce:

The following doesn't work normally.

$\DeclarePairedDelimitersXPP\norm[2]{}{\lVert}{\rVert}{_{#1}}{#2}$
$\norm{2}{2}$

Technical details:

  • MathJax Version: 3.2.0
  • Client OS: Mac OS X 12.01
  • Browser: Safari 15.1

Supporting information:

I believe this is related to this implementation.
Body, pre, post should all be substituted with arguments.

@dpvc dpvc added Accepted Issue has been reproduced by MathJax team Ready for Development v3 labels Jan 7, 2022
@dpvc
Copy link
Member

dpvc commented Jan 7, 2022

Thanks for the report. You are correct about the location of the issue, and that pre and post should also include substitution just as body does.

@dpvc dpvc self-assigned this Jan 7, 2022
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

2 participants