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

Clarify the primary entry page #389

Merged
merged 5 commits into from
Dec 19, 2018
Merged

Clarify the primary entry page #389

merged 5 commits into from
Dec 19, 2018

Conversation

mattgarrish
Copy link
Member

@mattgarrish mattgarrish commented Dec 19, 2018

This PR addresses two of the concerns raised in issue #386:

  • more clearly state that the PEP is a required resource
  • remove the parenthetical about what content the PEP might contain

Preview | Diff

Copy link
Contributor

@llemeurfr llemeurfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the sentence "If a default reading order is not provided, however, user agents will use the primary entry page as the default entry." makes me wonder: what if the reading order is provided? isn't the PEP the default entry anymore? this is the redundency of the term "(default) entry" which is the problem. Could we replace it by something like "if ..., user agents will consider that the primary entry page is the unique entry in the reading order"?

@HadrienGardeur
Copy link

It's a fallback, so we shouldn't be talking about a "default entry" IMO.

It's meant to make things easier for "single resource in the reading order" publications.

@mattgarrish
Copy link
Member Author

It's a fallback, so we shouldn't be talking about a "default entry" IMO.

Right, it's just trying to capture that it gets inserted when creating the canonical manifest if no reading order is specified. I've removed the default entry aspect and linked it to the canonical manifest step.

@iherman
Copy link
Member

iherman commented Dec 19, 2018

The latest commit from @mattgarrish is in line with the text in 4.5.1:

The default reading order is specified directly in the manifest, but MAY be omitted when it only consists of the primary entry page . When the default reading order is absent, user agents MUST include an entry for the primary entry page when compiling the canonical manifest.

+1 thus

@iherman
Copy link
Member

iherman commented Dec 19, 2018

Let us note, however, that this PR does not solve issue #386. It does not say that the PEP should be within the bounds of the publication. (Which is all right for this PR, it is up to us to drive #386 to a conclusion...)

@mattgarrish
Copy link
Member Author

It does not say that the PEP should be within the bounds of the publication.

Isn't that required now? If the WP MUST have a PEP, then it has to be specified either in the resources or the default reading order, which puts it within the bounds.

Would it help to add a parenthetical to this effect after the first MUST, like:

Every Web Publication MUST include a primary entry page (i.e., this resource has to be listed in the default reading order or resources),

@iherman
Copy link
Member

iherman commented Dec 19, 2018

It does not say that the PEP should be within the bounds of the publication.

Isn't that required now? If the WP MUST have a PEP, then it has to be specified either in the resources or the default reading order, which puts it within the bounds.

Where does it say that? I was looking at 3.4, and I did not see this stated. It just says it is a "resource" (in its general form), but that may mean it is part of the "links" array...

So yes, I believe this should be explicitly stated.

@mattgarrish
Copy link
Member Author

I modified the first paragraph of 3.4 to use MUSTs, so it now reads:

Every Web Publication MUST include a primary entry page, which is the resource that MUST be returned when accessing the Web Publication's address. The primary entry page represents the preferred starting resource for the Web Publication, and enables discovery of the manifest.

Or that's what you should be seeing if the preview isn't doing something weird.

If we want it clearer, I can add a parenthetical like I mentioned above.

@iherman
Copy link
Member

iherman commented Dec 19, 2018

@mattgarrish that is what I see indeed. And it does not say that, formally, it must be part of either the reading order or the resources array in the manifest, and that is what defines the boundaries...

I think adding the parenthetical remark is actually necessary.

@mattgarrish
Copy link
Member Author

mattgarrish commented Dec 19, 2018

I've rewritten the paragraph, as we're starting to repeat statements elsewhere in the specification. In the immediately preceding Resources section, for example, we say:

A Web Publication MUST include at least one HTML document [html]—the primary entry page.

Instead of repeating that again, I've moved the definition of what the PEP is to the start and made the only MUST that it has to be in one of those lists. The address section also states that it is equivalent of the PEP, so I don't think we need to normatively say again that is the expectation. For quick reference, it's now:

The primary entry page represents the preferred starting resource for a Web Publication and enables discovery of its manifest. It is the resource that is returned when accessing the Web Publication's address, and MUST be included in either the default reading order or the resource list.

Copy link
Contributor

@TzviyaSiegman TzviyaSiegman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With comments and changes already address, LGTM. Thanks!

@iherman
Copy link
Member

iherman commented Dec 19, 2018

As far as I am concerned it is a go!

@toshiakikoike
Copy link

Is it not necessary to define the primary entry page in the manifest?

https://w3c.github.io/wpub/#simple-book
C.1. Simple Book
There are no definition of primary entry page in either "readingOrder" or "resources".

Or is the HTML pointed to by "url" (by default "./index.html") treated as a primary entry page?
In other words, in the "Simple Book" sample,
is "https://publisher.example.org/mobydick/index.html" treated as a primary entry page?

@mattgarrish
Copy link
Member Author

Or is the HTML pointed to by "url" ... treated as a primary entry page?

Yes.

@mattgarrish
Copy link
Member Author

To respond a little more completely (was travelling earlier), the url property has to reference the primary entry page. As noted in the recent tag review, there are a couple of paragraphs that describe what the address should reference, but those are outdated and needed to be removed in this pull request. They will be removed in a pending update.

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

Successfully merging this pull request may close these issues.

8 participants