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.

Sign up for GitHub

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

Jump to bottom

Empty lists not allowed in navigation document #889

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

Empty lists not allowed in navigation document #889

mattgarrish opened this issue Nov 12, 2018 · 0 comments
Assignees
@rdeltour
Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: completed Work completed, can be closed
Milestone
4.2.0-alpha-1

Comments

@mattgarrish
Copy link
Member

In the navigation document, ol elements inside a nav must have at least one child li.

This restriction only applies to nav elements with an epub:type attribute.

Reference: https://w3c.github.io/publ-epub-revision/epub32/spec/epub-packages.html#sec-package-nav-def-model
@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
@rdeltour rdeltour self-assigned this Jan 10, 2019
@rdeltour rdeltour added status: in progress The issue is being implemented by the development team and removed status: accepted Ready to be further processed labels Jan 10, 2019
@rdeltour rdeltour added this to the 4.2.0-alpha-1 milestone Jan 10, 2019
rdeltour added a commit that referenced this issue Jan 14, 2019
@rdeltour
test: add a test for empty lists in nav doc
e760093 
Closes #889
@rdeltour rdeltour mentioned this issue Jan 14, 2019
Merged
@rdeltour rdeltour added status: completed Work completed, can be closed and removed status: in progress The issue is being implemented by the development team labels Jan 14, 2019
rdeltour added a commit that referenced this issue Jan 14, 2019
@rdeltour
test: add a test for empty lists in nav doc
232281c 
Closes #889
rdeltour pushed a commit that referenced this issue Feb 19, 2019
@mattgarrish @rdeltour
feat: update recognized font Core Media Types
c9bcd93 
- `CSS-006` and `CSS-017` that report position fixed/absolute are now
  `USAGE` messages
- Removes `CSS-027` as it would be a duplicate of `CSS-017`
- Command line severity tests now use a missing title as the `WARNING`
  trigger (instead of the now usage-only absolute positionning)

Fix #889
rdeltour pushed a commit that referenced this issue Feb 19, 2019
@mattgarrish @rdeltour
feat: report CSS absolute/fixed positioning as USAGE
672ac6d 
- `CSS-006` and `CSS-017` that report position fixed/absolute are now
  `USAGE` messages (instead of `WARNING`)
- Removes `CSS-027` as it would be a duplicate of `CSS-017`
- Command line severity tests now use a missing title as the `WARNING`
  trigger (instead of the now usage-only absolute positionning)

Fix #889
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdeltour rdeltour

Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: completed Work completed, can be closed
Projects
None yet
Milestone
4.2.0-alpha-1
Development

No branches or pull requests
2 participants
@rdeltour @mattgarrish