-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add CHTML Output Processor #43
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! 🎊
- I’d prefer casing acronyms as normal words:
CHTMLRenderer
->ChtmlRenderer
,fontURL
->fontUrl
,rehypeMathJaxSVG
->rehypeMathJaxSvg
; what do you think? - I think it makes more sense to default to one from
index.js
instead of exporting both. Exporting both is probably not what you want. We should pick the one that makes the most sense and export that fromindex.js
.
I updated symbol names, however, we should not change the symbol |
According to the documet of MathJax,
There is no recommendation of output format, every output format treated equally in document. |
Yeah, seems like SVG is a good default, as it works without options! |
Closes GH-42
I add two plugins
rehypeMathJaxCHTML
andrehypeMathJaxSVG
instead ofrehypeMathJax
.Users can choose the output processor of MathJax by this change.
API
options
: MathJax ouput options for CommonHTML ouput processor and SVG ouput processor