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

Math macro in jinja templates. #4703

Merged
merged 4 commits into from Jan 13, 2014
Merged

Math macro in jinja templates. #4703

merged 4 commits into from Jan 13, 2014

Conversation

damianavila
Copy link
Member

With the aim of reuse code, I have written a mathjax jinja macro living inside math.tpl.
In this way, the two templates html_full and slides_reveal (an any other) can call this macro (a sort a function) using the {{ mathjax() }} after importing the macro.

@Carreau
Copy link
Member

Carreau commented Dec 18, 2013

LGTM, I alway like commits that do the same with less line of code.

@damianavila
Copy link
Member Author

Somebody else?

@damianavila
Copy link
Member Author

And in this one... any other comment?

@minrk
Copy link
Member

minrk commented Jan 12, 2014

needs rebase. My only comment is that it seems like the mathjax macro template should be called mathjax, not math.

@damianavila
Copy link
Member Author

Rebased and changes done accordingly with @minrk comments...

minrk added a commit that referenced this pull request Jan 13, 2014
Mathjax macro in jinja HTML templates.
@minrk minrk merged commit fdb1b7e into ipython:master Jan 13, 2014
@damianavila damianavila deleted the math_macro branch January 13, 2014 20:53
@damianavila
Copy link
Member Author

Thanks!

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
Mathjax macro in jinja HTML templates.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants