Skip to content

Add LuaMML sockets for eqnarray#1817

Draft
zauguin wants to merge 2 commits intodevelopfrom
eqnarray-luamml
Draft

Add LuaMML sockets for eqnarray#1817
zauguin wants to merge 2 commits intodevelopfrom
eqnarray-luamml

Conversation

@zauguin
Copy link
Member

@zauguin zauguin commented Jul 6, 2025

Not entirely sure where to add these. So we want to keep this in latex-lab or should the sockets be added directly in base/ltmath?

Internal housekeeping

Status of pull request

  • Feedback wanted

Checklist of required changes before merge will be approved

  • Test file(s) added
  • Version and date string updated in changed source files
  • Relevant \changes entries in source included
  • Relevant changes.txt updated
  • Rollback provided (if necessary)?
  • ltnewsX.tex (and/or latexchanges.tex) updated

@zauguin zauguin requested a review from u-fischer July 6, 2025 13:46
@mbertucci47
Copy link
Contributor

mbertucci47 commented Jul 6, 2025

I think something more has to be done for fleqn.clo since that overwrites this definition.

@u-fischer
Copy link
Member

When I activate structure elements, the labels are not really correct as they are outside of the table.

\DocumentMetadata
  {
    uncompress,
    tagging=on,
    tagging-setup={math/setup=mathml-SE},
  }
\documentclass{article}

\usepackage{unicode-math}

\begin{document}

\begin{eqnarray}
A&=&B\\
D&=&C
\end{eqnarray}

\end{document}

I get also warnings:

Package tagpdf Warning: structure with label 0 is unknown rerun


Package tagpdf Warning: structure with label 0 is unknown rerun

(I tried also with the luamml-dev).

@zauguin
Copy link
Member Author

zauguin commented Jul 8, 2025

@u-fischer I forgot to push, but with latex3/luamml@732e0fb the structure tree looks reasonable.

Copy link
Member

@u-fischer u-fischer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It needs a luamml update and then it is fin imho

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

This PR has been automatically marked as stale because it has not had recent activity.

@github-actions github-actions bot added the stale label Sep 9, 2025
@josephwright
Copy link
Member

What are we doing with this PR?

@github-actions github-actions bot removed the stale label Nov 12, 2025
@FrankMittelbach
Copy link
Member

well, as @mbertucci47 remarked it also needs handling in fleqn. Not checked that.

@mbertucci47
Copy link
Contributor

I think the fleqn support was added in 4d98d8a

@github-actions
Copy link

This PR has been automatically marked as stale because it has not had recent activity.

@josephwright
Copy link
Member

@zauguin Ping - status here?

@github-actions github-actions bot removed the stale label Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants