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

Explainer contradicting the spec regarding doctype declaration #15

Closed
Malvoz opened this issue Jun 8, 2020 · 5 comments
Closed

Explainer contradicting the spec regarding doctype declaration #15

Malvoz opened this issue Jun 8, 2020 · 5 comments

Comments

@Malvoz
Copy link
Member

Malvoz commented Jun 8, 2020

https://github.com/Maps4HTML/MapML-Proposal#the-textmapml-mime-type

The proposed doctype for such a map document is <!doctype mapml>

https://maps4html.org/MapML/spec/#documents

A MapML document is a [MicroXML] document. As such, it MUST have a single root element named <mapml>. It MUST NOT have a DOCTYPE declaration

@prushforth
Copy link
Member

Yes, that's a legit discrepancy / contradiction. The reason the discrepancy exists is that I began developing MapML from a theory that MicroXML syntax was compatible with the HTML parser. I am not / no longer certain that this is true.

At a minimum, I think I should have put MapML content into the XHTML namespace, which seems to be used by the HTML parser internally for sorting HTML elements from other stuff; in that case it would mean that a MapML document is not/would not be a MicroXML document. It's (more) important to be compatible with the HTML parser than to be compatible with MicroXML.

However, since @zcorpan is going to review the spec/proposal from a parser implementer perspective, I hope we'll be able to resolve this issue with Simon's help.

@zcorpan
Copy link

zcorpan commented Jun 9, 2020

@prushforth I'm not a member of this GitHub org yet, so I can't self-assign.

I'm planning to start working on the MapML review next week.

@prushforth
Copy link
Member

@zcorpan I sent you an invitation. Cheers

@Malvoz
Copy link
Member Author

Malvoz commented Jul 9, 2020

This should be clarified in Maps4HTML/MapML-Specification#72. Keeping this issue open until a conclusion has been made there, as a reminder to update the text if need be.

@prushforth
Copy link
Member

The spec has been updated to remove MicroXML references, and the intention is still to eventually support an HTML Parser-compatible <!doctype mapml>, so closing this as resolved, and Maps4HTML/MapML-Specification#72 can stay open.

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

No branches or pull requests

3 participants