readium / r2-testapp-swift

BSD 3-Clause "New" or "Revised" License
146 stars 38 forks source link

LCP: issue with ToC #264

Closed llemeurfr closed 5 years ago

llemeurfr commented 5 years ago

Licenses taken from front-prod.edrlab.org.

On Moby-Dick, the ToC is empty. On Waste Land the Toc is present but does not work.

mickael-menu-mantano commented 5 years ago

The empty toc on moby-dick is fixed by this PR https://github.com/readium/r2-streamer-swift/pull/105

For Wasteland, the scrollToId function in utils.js doesn't work (I tried using it directly in the web inspector). But it hasn't been touched for 2 years so it might be a problem specifically with this book.

mickael-menu-mantano commented 5 years ago

I found a solution and documented it in an issue on r2-streamer. There's already too much PR in the waiting list and I want to avoid merging pain. https://github.com/readium/swift-toolkit/issues/99