Skip to content

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