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

Deal with empty content nodes #29

Open
christianp opened this issue Apr 11, 2014 · 5 comments
Open

Deal with empty content nodes #29

christianp opened this issue Apr 11, 2014 · 5 comments
Labels

Comments

@christianp
Copy link
Contributor

http://www.w3.org/Math/testsuite/build/main/ErrorHandling/NumChildren/noChildContent-form.xhtml

These should put some kind of error tag in.

@christianp christianp added the bug label Apr 11, 2014
christianp added a commit that referenced this issue May 2, 2014
see #29.

Currently the message is "missing child node", which isn't particularly helpful.
@pkra
Copy link

pkra commented May 2, 2014

I think this test reveals a regular MathJax bug with empty annotation tags.

@davidcarlisle
Copy link

On 2 May 2014 10:59, Peter Krautzberger [email protected] wrote:

I think this test reveals a regular MathJax bug with empty annotation tags.

The test is wrong.

annotation and annotation-xml are both optional inside semantics and you
can have as many as you want in either order, but there should be an
initial element (which is not annotation or annotation-xml) which is the
thing being annotated so the last two expressions in that test

and

/semantics>

are both in error but the test shows one as an error and one not, which I
don't think is justified.
(although rendering both as rather than an error is probably
reasonable

@pkra
Copy link

pkra commented May 2, 2014

Thanks for the correction, @davidcarlisle. Either way, it's not really something ctop.js needs to concern itself with, wouldn't you say?

@christianp
Copy link
Contributor Author

Yep. I've just made it so the code doesn't completely fail when a child node is missing.

@davidcarlisle
Copy link

On 2 May 2014 12:48, Christian Perfect [email protected] wrote:

Yep. I've just made it so the code doesn't completely fail when a child
node is missing.

sounds like a good plan:-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants