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

span elements within nav elements #824

Closed
murata2makoto opened this issue Aug 25, 2016 · 5 comments
Closed

span elements within nav elements #824

murata2makoto opened this issue Aug 25, 2016 · 5 comments
Labels
EPUB32 Issues from 3.0.1 resolved in the EPUB 3.2 specification Topic-PackageDoc The issue affects package documents
Milestone

Comments

@murata2makoto
Copy link
Contributor

http://www.idpf.org/epub/31/spec/epub-packages.html#confreq-nav-a

Each list item of the ordered list represents a heading, structure or other item of interest, and must contain either a child a element or a child span element. The a element describes the target that the link points to. The span element serves as a heading for breaking down lists into distinct groups (for example, a large list of illustrations can be segmented into several lists, one for each chapter).

Is <nav epub:type="toc"><ol><li><span>foo</span></li></ol></nav> allowed? In other words, can we have a <span>without sibling <a> elements or <ol> elements?

@rdeltour
Copy link
Member

rdeltour commented Aug 25, 2016

Is <nav epub:type="toc"><ol><li><span>foo</span></li></ol></nav> allowed? In other words, can we have a <span> without sibling <a> elements or <ol> elements?

No. A following bullet says:

The span element must be followed by an ol ordered list; it cannot be used in "leaf" li elements.

@murata2makoto
Copy link
Contributor Author

Yes, you are right. The sentence you quoted disallows my example shown above.

But I think that we have to be more explicit. I was confused, and I know that one implementor was confused too.

nav/ol/li can have either an a element possibly followed by a single ol element, or a span element followed by a single ol element. Other than whitespace, no text chunk is allowed to occur as a child of nav/ol/li.

@mattgarrish
Copy link
Member

I don't see more prose is going to make it easier to read.

A content model box to the start of the section would probably be better, like this:

nav

@mattgarrish mattgarrish modified the milestone: EPUB 3.1 Aug 25, 2016
@mattgarrish mattgarrish added the Topic-PackageDoc The issue affects package documents label Aug 25, 2016
@mattgarrish
Copy link
Member

I think we can also remove the duplicative content model prose if we do this. See a possible rewrite here:
https://docs.google.com/document/d/1l96NqIYEun1dzuepGELsfmnbwucLv4nIKfoWlWJWUds/edit#heading=h.wuwozrqhc3jm

@mattgarrish
Copy link
Member

Proposed Solution

Add a content model box and update the prose as suggested in the referenced google doc.

@mattgarrish mattgarrish added the Status-Proposed Solution A proposed solution has been included in the issue for working group review label Aug 26, 2016
mattgarrish added a commit that referenced this issue Sep 1, 2016
#755 - change alt-script to alt-rep and clarify language
#761 - make image cmts required when there is a viewport
#773 - update roadmap and add diagram
#778 - clarify package conformance
#780 - generalize backwards compatibility statement
#800 - clarify svg handling for fxl documents
#808 - replace spaces with underscores in rootfile examples
#822 - fix obsolete feature labels/descriptions
#823 - add note about incomplete RS requirements for scrolled-continuous
#824 - add clearer content model for nav elements
#826 - note toc nav is required in intro
#828 - clarify ordering requirements for toc nav references
#829 - note optional use of pagebreak with page-list

adds a link to the informative a11y faq;
patches errata not applied to doi examples;
probably some other minor stuff, too
@mattgarrish mattgarrish removed the Status-Proposed Solution A proposed solution has been included in the issue for working group review label Sep 1, 2016
@mattgarrish mattgarrish added the EPUB32 Issues from 3.0.1 resolved in the EPUB 3.2 specification label Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EPUB32 Issues from 3.0.1 resolved in the EPUB 3.2 specification Topic-PackageDoc The issue affects package documents
Projects
None yet
Development

No branches or pull requests

3 participants