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

Added MathML input. #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chris-malloy-mri
Copy link

No description provided.

@lgeiger
Copy link
Member

lgeiger commented Apr 25, 2017

Thanks a lot for looking into this!

I see you included the default MathML_SVG_full config. That's fine in a default MathJax environment, but I imagine this config also requires some features we don't support (since it we don't ship every mathjax file). Would you be able make a custom config from scratch, similar to our electron one?

Also it would be really awesome if you could document the usage in the readme.

@chris-malloy-mri
Copy link
Author

By the way, is there any reason you went with SVG instead of CommonHTML for this?

@gervaisb
Copy link

Hello. Do you have any plan to merge this PR ?

@rgbkrk
Copy link
Member

rgbkrk commented Oct 11, 2018

There's a hearty discussion + outlined reasoning in #30 (comment) and we've started moving this package forward again.

We've tried to keep this trimmed down, we're not sure how small we can make the MathML variant. There may come a day that we need to publish multiple packages each with separate builds. That might be the only way to get what each group needs.

@rgbkrk
Copy link
Member

rgbkrk commented Oct 11, 2018

How can I learn more about what compels folks to use MathML instead of latex?

@chris-malloy-mri
Copy link
Author

chris-malloy-mri commented Oct 11, 2018 via email

@lgeiger
Copy link
Member

lgeiger commented Oct 11, 2018

Main reason would be MathML is supported by screen readers.

Even if only used as input while still rendering with CommonHTML?

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.

4 participants