dauwhe / epub31-bff

Straw-man spec for browser-friendly format for EPUB31
15 stars 3 forks source link

Nesting sub-collections between links. #20

Open dauwhe opened 8 years ago

dauwhe commented 8 years ago

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 :)

TzviyaSiegman commented 8 years ago

Remember that example 40 in indexing is invalid (see https://github.com/IDPF/epub-revision/issues/561)

dauwhe commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

Hopefully better now. Thanks!

HadrienGardeur commented 8 years ago

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