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

Use of display-seq meta property is deprecated #877

Closed
mattgarrish opened this issue Nov 12, 2018 · 4 comments
Closed

Use of display-seq meta property is deprecated #877

mattgarrish opened this issue Nov 12, 2018 · 4 comments
Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: completed Work completed, can be closed

Comments

@mattgarrish
Copy link
Member

Epubcheck should emit a warning if the property is used in the meta property attribute.

Reference: https://w3c.github.io/publ-epub-revision/epub32/spec/epub-packages.html#sec-display-seq

@mattgarrish mattgarrish added spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: accepted Ready to be further processed labels Nov 12, 2018
mattgarrish added a commit that referenced this issue Dec 17, 2018
- partially resolves #883 - only adds the attribute, doesn't check values
- fixes #882 - reports if a preceding itemref has a matching idref
- fixes #881 - reports presence of bindings
- fixes #880 - reports presence of rendition:viewport property in metadata
- fixes #879 - reports rendition:spread=portrait in metadata and rendition:spread-portrait on an itemref
- fixes #878 - reports presence of meta-auth property in metadata
- fixes #877 - reports presence of display-seq in metadata
- fixes #876 - reports if term/authority are not used together, or if multiple instances are attached to a single subject
rdeltour pushed a commit that referenced this issue Jan 14, 2019
- partially resolves #883 - only adds the attribute, doesn't check values
- fixes #882 - reports if a preceding itemref has a matching idref
- fixes #881 - reports presence of bindings
- fixes #880 - reports presence of rendition:viewport property in metadata
- fixes #879 - reports rendition:spread=portrait in metadata and rendition:spread-portrait on an itemref
- fixes #878 - reports presence of meta-auth property in metadata
- fixes #877 - reports presence of display-seq in metadata
- fixes #876 - reports if term/authority are not used together, or if multiple instances are attached to a single subject
rdeltour pushed a commit that referenced this issue Jan 14, 2019
- partially resolves #883 - only adds the attribute, doesn't check values
- fixes #882 - reports if a preceding itemref has a matching idref
- fixes #881 - reports presence of bindings
- fixes #880 - reports presence of rendition:viewport property in metadata
- fixes #879 - reports rendition:spread=portrait in metadata and rendition:spread-portrait on an itemref
- fixes #878 - reports presence of meta-auth property in metadata
- fixes #877 - reports presence of display-seq in metadata
- fixes #876 - reports if term/authority are not used together, or if multiple instances are attached to a single subject
@murata2makoto
Copy link
Contributor

The warning message is fine but epubcheck 4.2.0-alpha-1 reports that EPUB 3.0.1 rules are used. Is this specific to the JP locale?

@rdeltour
Copy link
Member

epubcheck 4.2.0-alpha-1 reports that EPUB 3.0.1 rules

doh, good catch! 😅

Is this specific to the JP locale?

No, the EPUB "3.x" version string just hasn't been updated, it will be reported the same on all locales! oops…

@rdeltour
Copy link
Member

I created #943 to make sure we don't forget 😄

@rdeltour rdeltour added status: completed Work completed, can be closed and removed status: accepted Ready to be further processed labels Jan 21, 2019
@rdeltour
Copy link
Member

Closed in #927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: completed Work completed, can be closed
Projects
None yet
Development

No branches or pull requests

3 participants