-
-
Notifications
You must be signed in to change notification settings - Fork 275
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
Move luamml patches into latex-lab #1666
Conversation
… into UF-latex-lab-luamml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so far only review on the doc ... have to take a break for lunch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a check over the line-ends would be sensible - outside of expl3
syntax, I'd prefer to see them ending with %
where a space can creep in, even if used in math mode. But as this is code being moved from elsewhere, I've gone for 'approve' not 'request changes'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
latex-lab-amsmath
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
review -math and -mathtools
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few more files looked at
required/latex-lab/math-tagging-examples/math-17-alt+mathmlAF+texAF.tlg
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming the various comments are resolved (or rejected)
This moves the various patches from luamml into the latex-lab code and replaced them with proper sockets.
The various amsmath patches already present in latex-lab moved into latex-lab-amsmath.
A number of bugs have been fixed in the process, and with luatex more luatex primitives are used (e.g. \mathstyle in \mathpalette).
The split environment isn't properly tagged yet, but a (partly) fix is comming.
The default pdfversion has been set to 2.0 with \DocumentMetadata.
Internal housekeeping
Status of pull request
Checklist of required changes before merge will be approved
\changes
entries in source includedchanges.txt
updatedltnewsX.tex
(and/orlatexchanges.tex
) updated