Widelands Website

Merge lp:~widelands-dev/widelands-website/mathjax_for_documentation into lp:widelands-website

  1. mathjax_for_documentation
  2. Merge into trunk
Proposed by kaputtnik
Status: Merged
Merged at revision: 464
Proposed branch: lp:~widelands-dev/widelands-website/mathjax_for_documentation
Merge into: lp:widelands-website
Diff against target: 15 lines (+6/-0)
1 file modified
templates/sphinxdoc/base.html (+6/-0)
To merge this branch: bzr merge lp:~widelands-dev/widelands-website/mathjax_for_documentation
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+326800@code.launchpad.net

Description of the change

Fix for displaying formulas when switching to mathjax for the sphinx documentation.

Should be merged if https://code.launchpad.net/~widelands-dev/widelands/codecheck/+merge/326647 is merged.

To post a comment you must log in.
Revision history for this message
SirVer (sirver) wrote :

Thanks! There is no harm in merging this right-away, it is not user visible unless mathjax is actually used.

review: Approve
Revision history for this message
kaputtnik (franku) wrote :

Yes, you're right :-)

merged and deployed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/sphinxdoc/base.html'
2--- templates/sphinxdoc/base.html 2015-02-18 22:30:08 +0000
3+++ templates/sphinxdoc/base.html 2017-07-04 19:16:39 +0000
4@@ -6,6 +6,12 @@
5
6 {% block extra_head %}
7 <link rel="stylesheet" type="text/css" media="all" href="{{ MEDIA_URL }}css/documentation.css" />
8+
9+{# using mathjax for formulas #}
10+<script type="text/javascript" async
11+ src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_CHTML">
12+</script>
13+
14 {{ block.super}}
15 {% endblock %}
16

Subscribers

People subscribed via source and target branches