-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
Ltnews40 intro, corrections and some reordering #1512
Conversation
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 typo
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.
Looks good, apart from two typos one comment:
This new feature can be disabled with \verb+\tagpdfsetup{math/mathml/luamml=false}+
This should change if the math-PR is merged/reviewed. To fit in the structure elements the key names have changed (this one still continues to work for now but is not documented). There is a summary section in latex-lab-math which tries to show all keys, and the examples show all main keys too.
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.
some typos and rewordings done as code diff suggestions so they can be accepted (or not) on site
Then please provide me with a new text, as the math-PR assuming that the math-PR gets merged (which I think it should). |
Co-authored-by: David Carlisle <d.p.carlisle@gmail.com> Co-authored-by: Skillmon <Skillmon@users.noreply.github.com>
Test on Update: But for now, for unknown reason I cannot reproduce the .tlg update locally. |
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.
Minot typo-level suggestions
Co-authored-by: Joseph Wright <joseph@texdev.net>
We now good-to-go here? |
@u-fischer suggested we should change one description once the latex-lab-math is merged (for that we need text) and and there is the question about use of "efficiencies" other than fine from my side. |
ah I see you changedthe latter already. |
I would suggest to say that the loading of luamml can be suppressed with |
ok done |
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
) updatedLooks like the ltugboat change hasn't yet made it to all mirrors, so at the moment the firstaid test file errors (after David's change to it). Not much that one can do about it other that wait, I guess.
Looks like the pull request ran through, so it is really an issue that will go away by tomorrow (hopefully).