Make Mathjax rendering optional in shortanswers #2738
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If an answer is not supposed to involve mathematics, rendering it with MathJax and displaying the result is at best confusing and at worst will actually display something that makes no sense.
This adds a switch to the
<response>to disable mathjax rendering and a book level switch to change the default behavior. Otherwise, the default is to render with MathJax in order to maintain existing behavior.Note that you should see the change in the HTML output, but RS will still render MathJax until this PR is in place:
RunestoneInteractive/rs#1082
To test without a RS server, make sure to use stringparam
short-answer-responses always.Tagging @bnmnetp