search

w3c / wpub

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

Why is resource list required? #311

Closed dauwhe closed 6 years ago

dauwhe commented 6 years ago

resource-list is a required property for a WPUB manifest.

How does this actually work? We don't require any particular component of a web publication to be in this list. What if a WP consisted only of HTML files that were in the default reading order? Would you then be required to have an empty resource list?

HadrienGardeur commented 6 years ago

I agree, it shouldn't be a required property (it isn't required in the current JSON Schema).

iherman commented 6 years ago

To be honest, I have no idea how it ended there, and I do not see any reason for it either.

ghost commented 6 years ago

It looks like inherited from epub? Maybe add some note to mention exception is when WP consisted only of items that is already in default reading order?

iherman commented 6 years ago

It looks like inherited from epub?

Ah. That may be the case

Maybe add some note to mention exception is when WP consisted only of items that is already in default reading order?

The draft currently says that:

The resource list enumerates all resources that are used in the processing and rendering of a Web Publication (i.e., that are within its bounds) and that are not listed in the default reading order.

With that, I am not sure having that exception mentioned really makes sense.

I vote to remove it from the required list.

ghost commented 6 years ago

Yes I missed this line. It seems conflicted with Required. I agree with removing it.

mattgarrish commented 6 years ago

It looks like inherited from epub?

Not so much this as that this:

and that are not listed in the default reading order.

was added later. Earlier on, it was an open issue whether resources in the reading order also had to be in resource list.

But a no-brainer change now, if there ever was one.

TzviyaSiegman commented 6 years ago

@mattgarrish make it so!

mattgarrish commented 6 years ago

Closing this with the commit of #313