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

V3: activating accessibility results in "Math input error" #2255

Closed
NSoiffer opened this issue Dec 5, 2019 · 9 comments
Closed

V3: activating accessibility results in "Math input error" #2255

NSoiffer opened this issue Dec 5, 2019 · 9 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Needed v3.0

Comments

@NSoiffer
Copy link

NSoiffer commented Dec 5, 2019

Visit the page https://mathjax.github.io/MathJax-demos-web/mml-chtml.html. It renders well. Right click on some math, choose Accessibility:Activate. The math is now replaced by "Math input error". Here's a small snippit:
image

The page's heading says "...v3 beta...", but it appears to link to the V3 release. Here's the script tags from the page:

<script src="https://polyfill.io/v3/polyfill.min.js?features=es6"></script>
--
  | <script id="MathJax-script" async src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/mml-chtml.js"></script>
@dpvc dpvc added the Accepted Issue has been reproduced by MathJax team label Dec 5, 2019
@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

Thanks, I can confirm the behavior. I'll check into it.

@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

I'll also update the page titles. Thanks!

@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

Note that reloading the page displays properly and has the accessibility features activated. So the problem is somewhere in the updating of the page to add/remove accessibility.

@zorkow
Copy link
Member

zorkow commented Dec 5, 2019

This only happens for MathML input, i.e., mml-svg.html and mml-chtml.html pages. TeX works fine.

Just FYI: The issue is filed in the wrong repo.

@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

Just FYI: The issue is filed in the wrong repo.

@zorkow, I don't agree. What repository did you have in mind?

@zorkow
Copy link
Member

zorkow commented Dec 5, 2019

mathjax-src

@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

No, we agreed that all issues for MathJax would be tracker here, so that they were easier to search and would all be in one location. This is what is listed in the documentation, and I've certainly been pointing people here.

@zorkow
Copy link
Member

zorkow commented Dec 5, 2019

Ah, I had recalled it exactly the other way around (i.e,. that we wanted to forward with this to the -src repo, automatically if possible). But looking at the doc, you're right, of course.

Well done @NSoiffer ! You've found the right place.

@dpvc
Copy link
Member

dpvc commented Dec 5, 2019

The decision of which to use came down to the fact that all the old v2 issues were here, so to keep everything in one place, this was the natural place to do it.

@dpvc dpvc self-assigned this Dec 30, 2019
dpvc added a commit to mathjax/MathJax-src that referenced this issue Jan 11, 2020
…or rerender(). Save processed when menu creates a new document, and dont' actually recompile on rerender (we just need enrichment, etc.). Resolves issue mathjax/MathJax#2255.
@dpvc dpvc removed their assignment Jan 12, 2020
dpvc added a commit to mathjax/MathJax-src that referenced this issue Jan 30, 2020
Fix problem with enabling explorer with MathML input.  mathjax/MathJax#2255
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Jan 30, 2020
@dpvc dpvc added this to the MathJax v3.0.1 milestone Feb 5, 2020
@dpvc dpvc added Fixed v3.0 and removed Merged Merged into develop branch labels Feb 10, 2020
@dpvc dpvc closed this as completed Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Needed v3.0
Projects
None yet
Development

No branches or pull requests

3 participants