futurepress / epub.js

Enhanced eBooks in the browser.
http://futurepress.org
Other
6.39k stars 1.09k forks source link

Fixed-Layout epubs and page-spread-* properties #1105

Open halmos opened 4 years ago

halmos commented 4 years ago

Thanks for your work on this project - we've been using it extensively and very much appreciate its ongoing development.

We've come accross an issue with fixed-layout in which epubjs does not appear to honor the spine properties for page-spread-* see: http://idpf.org/epub/fxl/#properties-page-spread

As a result, fixed layout epubs often have mis-aligned spreads.

Expected pagination:

[ ][R]
[L][R]
[L][R]
[L][ ]

Actual epubjs pagination

[R][L]
[R][L]
[R][L]

This a common issue with epubs exported from Adobe Indesign using fixed-layout conversion.

Other readers such as Apple's Books will show the initial page on its own and subsequent spreads are correctly aligned.

I noted the mention of a page offset in issue #1070 - Is there some other way to achieve this?

thank you again!

johnfactotum commented 4 years ago

The page-spread-* properties are already honored in 0.3.86 and after #927. So you probably just need to upgrade to the latest version. Though there is a bug with rendition.display(…), namely, #1070 and https://github.com/futurepress/epub.js/pull/927#issuecomment-629780838.

halmos commented 4 years ago

thank you! (and sorry for not finding that you've already fixed it. )

johnfactotum commented 4 years ago

Just to clarify…

sorry for not finding that you've already fixed it.

I didn't… I'm neither the maintainer of this library, nor the author of the fix. I'm just a random person who happen to saw your issue :)