Skip to content
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

WIP html_utils: improved MathML support #131

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kaplun
Copy link
Contributor

@kaplun kaplun commented Mar 25, 2015

Signed-off-by: Samuele Kaplun samuele.kaplun@cern.ch

* Enriches the list of recognized MathML tags following the reference
  available at:
  https://developer.mozilla.org/en-US/docs/Web/MathML/Element

* Introduces the possibility to simply HTML-escape everything but the
  specific MathML tags, thus allowing to preserve MathML.

Signed-off-by: Samuele Kaplun <samuele.kaplun@cern.ch>
@kaplun
Copy link
Contributor Author

kaplun commented Mar 25, 2015

@jalavik I will add tests to proof the new behavior.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 65.01% when pulling d18a1de on kaplun:mathml into 5120481 on inspirehep:master.

@jalavik
Copy link
Contributor

jalavik commented May 5, 2015

@kaplun Shall we integrate it already or do the enhanced tests?

@kaplun
Copy link
Contributor Author

kaplun commented May 19, 2015

Wow I forgot about this branch. I think we need still tests, because of:

  • Introduces the possibility to simply HTML-escape everything but the
    specific MathML tags, thus allowing to preserve MathML.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants