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

Nesting sub-collections between links. #20

Open
dauwhe opened this issue Mar 10, 2016 · 5 comments
Open

Nesting sub-collections between links. #20

dauwhe opened this issue Mar 10, 2016 · 5 comments

Comments

@dauwhe
Copy link
Owner

dauwhe commented Mar 10, 2016

See example 5, which is essentially Example 40 from the indexing spec.

This seems insanely complex. Do we want to support this? Note that nesting collections in JSON becomes even less readable than in XML, and one can spend many unhappy hours at jsonlint.com :)

@dauwhe dauwhe changed the title Nesting sub-collections in between links. Nesting sub-collections between links. Mar 10, 2016
@TzviyaSiegman
Copy link

Remember that example 40 in indexing is invalid (see w3c/epub-specs#561)

@dauwhe
Copy link
Owner Author

dauwhe commented Mar 16, 2016

Example 5 fixed to reflect issue noted above.

This makes things worse. As evidence, I offer the last few lines of the JSON file:

          }]
        }
      ]
    }]
  }
}

@HadrienGardeur
Copy link
Collaborator

Looks invalid to me, you can't have an "index-group" (a subcollection) in "links", it goes against our model which is:

collection
    metadata
    links
    subcollection (same definition as collection)

@dauwhe
Copy link
Owner Author

dauwhe commented Mar 17, 2016

Hopefully better now. Thanks!

@HadrienGardeur
Copy link
Collaborator

Can we close that issue and potentially even remove the example from our current document?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants