Page MenuHomePhabricator

<math chem>\ce{\overbrace{\ce{H2O}}}</math> emits raw mathml in client svg and native mathml mode
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

Mathoid:<br>
<math chem forcemathmode=mathml>\ce{\overbrace{\ce{H2O}}}</math>

MathJax-client SVG:<br>
<math chem forcemathmode=mathjax>\ce{\overbrace{\ce{H2O}}}</math>

What happens?:

Rendering shows raw mathml syntax instead of the formula:

Screenshot 2026-09-21 at 20.33.53.png (772×264 px, 37 KB)

Other information (browser name/version, screenshots, etc.):

Simplified example from https://en.wikipedia.org/w/index.php?title=Proton_exchange_membrane_electrolysis&oldid=1368994456

Event Timeline

A similar error happens in

:<math chem title="General scheme of the Stille reaction">
{\color{Blue}\ce{R^1-Sn(Alkyl)3}} + {\color{Red}\ce{R^2-X}} 
\ \ce{->[{\color{Green}\ce{Pd^0}}\text{ (catalytic)}][\text{ligand set}]} \ 
\overbrace{{\color{Blue}\ce{R^1}}\!-\!{\color{Red}\ce{R^2}}}^{coupled\ product} + {\color{Red}\ce{X}}\!-\!{\color{Blue}\ce{Sn(Alkyl)3}}</math>

Found in a commented out section at top of https://en.wikipedia.org/wiki/Stille_reaction

Krinkle triaged this task as High priority.Mon, Sep 21, 7:32 PM
TheDJ renamed this task from <math chem>\ce{\overbrace{\ce{H2O}}}</math> emits raw mathml in clinet svg and native mathml mode to <math chem>\ce{\overbrace{\ce{H2O}}}</math> emits raw mathml in client svg and native mathml mode.Tue, Sep 29, 4:51 PM
Physikerwelt changed the task status from Open to In Progress.Sun, Oct 4, 7:12 AM
Physikerwelt claimed this task.
Physikerwelt subscribed.

Checking relation to Parsoid T399226

It seems to be unrelated. This can be easily fixed in the grammar... just need to avoid infinit loops

Actually it matters if its \ce{\overbrace{\ce{H2O}}} or \ce{\overbrace{\ce {H2O}}}. So I need to have a second look

Change #1350658 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] WIP: Convert \ce and \pu in the parser only

https://gerrit.wikimedia.org/r/1350658

Change #1350822 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Improve test coverage of LocalChecker

https://gerrit.wikimedia.org/r/1350822

Change #1350857 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Merge the option usemhchemtexified into usemhchem

https://gerrit.wikimedia.org/r/1350857

Change #1350860 had a related patch set uploaded (by Physikerwelt; author: Physikerwelt):

[mediawiki/extensions/Math@master] Remove the unused chem grammar

https://gerrit.wikimedia.org/r/1350860

Change #1350822 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Improve test coverage of LocalChecker

https://gerrit.wikimedia.org/r/1350822

Change #1350857 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Merge mhchemtexified into mhchem

https://gerrit.wikimedia.org/r/1350857

Change #1350658 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Convert \ce and \pu in the parser only

https://gerrit.wikimedia.org/r/1350658

Change #1350860 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Remove the unused chem grammar

https://gerrit.wikimedia.org/r/1350860