-
Notifications
You must be signed in to change notification settings - Fork 62
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
Clarify core media types #1345
Clarify core media types #1345
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks clear to me. The only minor change I might suggest is s/compact/contract.
Since CSS is a core media type, I tried to check when the support of CSS is required. Neither 2. Conformance Requirements nor I am not mistaken, 4.3 CSS Style Sheets mandates the support of CSS even when the RS does not have a viewport. |
Section 2 covers this:
|
You are right. But I think that the sentence you quoted should mention CSS. |
(This is not part of the changed, ie, I could not comment directly...) The section on foreign resources say:
I am not sure I properly parse the first bullet items. I presume we refer to the fact that, e.g., the HTML |
Agree, but I'll open a separate PR for this so we don't mix issues. |
Ya, it refers to the embedding format - SVG or XHTML or whatever the foreign resource is included in. |
I've changed this to:
|
Picking up on #1341 (comment), this is my best attempt to expand on the perennially confusing nature of core media types.
Let me know if there's anything you don't agree with, or if any parts need further elaboration.
Feedback is always welcome beyond the reviewer list, too.
Preview | Diff