w3c / wpub

W3C Web Publications
https://w3c.github.io/wpub/
Other
78 stars 19 forks source link

Remove infoset #359

Closed mattgarrish closed 6 years ago

mattgarrish commented 6 years ago

This PR removes the "infoset" concept and replaces it with more emphasis on the authored v. canonical manifests.

Prior to this, we had infoset and manifest subsections for almost every property, whereas now the information has been merged and, in most cases, the manifest expressions moved up after the introductory paragraph.

As this change required a significant reread of the specification, I've also made a number of minor editorial tweaks along the way. These are generally insignificant, with the exception of the language and direction section. As I read through, I noticed that most of the global language info was specified before the subsection containing the property definitions (plus at least one para that looked like it belonged with the overrides). @iherman , please have a look and see if my shuffling still makes sense to you.

Otherwise, I think there's a diagram or two that will need updating, but I didn't try and fix those.

mattgarrish commented 6 years ago

Not sure why no preview option is showing up. Only thing I can think is because the diff is too large. Respec doesn't show me any issues.

iherman commented 6 years ago

I do not know why preview does not work either.

This URI seems to work to display the new version:

https://tinyurl.com/y9ds858k

iherman commented 6 years ago

I have committed the document with some minor changes.

mattgarrish commented 6 years ago

Yes, I had the same thought last night, but even if authored=canonical, you don't actually get canonical until the user agent has processed the authored manifest, correct? That was the distinction I was trying to draw, between the one that someone/something has produced versus the post-processed.

Would it help to add some more clarification to that effect?

iherman commented 6 years ago

Well, if authored=canonical then the canonicalization is an identity operation... But yes, it will still happen, in this sense it is not 100% the same.

I think adding a somewhat more verbose clarification would indeed be the best, rather than producing a concise and fully precise text...

mattgarrish commented 6 years ago

I added another para, but feel free to mod directly if you have other ideas.

iherman commented 6 years ago

Ok, let us see what others think. It works for me...

mattgarrish commented 6 years ago

Here's an alternative preview that doesn't break when you click internal links: https://cdn.staticaly.com/gh/w3c/wpub/remove-infoset/index.html

mattgarrish commented 6 years ago

Should we go ahead and merge this so we don't block the other open issues? (It's mostly shuffling, so if it can benefit from further clarification that shouldn't impact on what exists now.)

iherman commented 6 years ago

Yes.