Skip to content

Conversation

@poke1024
Copy link
Contributor

This change allows MathML mglyphs with embedded data to be correctly rendered. This is an important feature for some applications using images embedded in math formulas.

To test this PR's change in a Python 3 notebook, use the attached notebook and evaluate all three cells.

Without this PR, the third step's HTML will output nothing. With the CSS change in this PR, an arrow svg will be visible as expected.

The second step is necessary to reconfigure MathJax to accept `mglyph' tags with embedded data, which is by default disabled due to Jupyter's safe MathJax mode. I will address this in a separate PR.

mglyph.ipynb.zip

@gnestor
Copy link
Contributor

gnestor commented Aug 15, 2016

@poke1024 I can confirm that master currently will not render the SVG and that this PR will render it. In testing, I noticed that the less-watch script only recompiles LESS on a file change and not on npm run build:watch so I will submit a PR to fix that...

@Carreau @minrk Feel free to test as well but I think this is ready to merge.

@gnestor gnestor added this to the 5.0 milestone Aug 15, 2016
@gnestor gnestor added the bug label Aug 15, 2016
@minrk minrk merged commit b9292d3 into jupyter:master Aug 17, 2016
@minrk
Copy link
Member

minrk commented Aug 17, 2016

Thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants