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

Additional cleanup necessary after yesterdays merge #801

Closed
xml-project opened this issue Apr 24, 2019 · 1 comment

Comments

Projects
None yet
2 participants
@xml-project
Copy link
Contributor

commented Apr 24, 2019

Yesterday @ndw introduced our new context of content-types for documents to the working draft.

After reading it again in context, I think some additional cleanup is necessary:

  1. To my reading the newly introduced section 3.3 makes the following section 3.4 almost completely obsolete. Only the error condition at the end of 3.4 seems to be relevant now.

  2. Section 3.4 (1) says, that "image/svg+xml" is an XML mediatype, but 3.3 says it is not. I would argue to add image/svg+xml to the standard list of content types, XML expands to.

  3. Section 3.3 starts with

In some contexts (step inputs, step outputs, and inline elements, for example) ...

I think that the reference to "inline elements" is misleading here. To my understanding p:inline/@content-type specifies the actual media type of the document, so there is no list here, no asterisk is allowed and no expansion will take place.

Of you give me a go, I would try to make these changes. Otherwise, could you please tell me where I am wrong.

@ndw

This comment has been minimized.

Copy link
Contributor

commented Apr 24, 2019

  1. Yes, you're probably right.
  2. Yes, I think xml should expand to: application/xml text/xml */*+xml -application/xhtml+xml
  3. Yes, that's just a brain cramp on my part.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.