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

Support PDF and ePub 2/3 rendering as HTML #68

Closed
Tracked by #74
anjackson opened this issue Sep 7, 2021 · 3 comments
Closed
Tracked by #74

Support PDF and ePub 2/3 rendering as HTML #68

anjackson opened this issue Sep 7, 2021 · 3 comments
Assignees

Comments

@anjackson
Copy link
Contributor

anjackson commented Sep 7, 2021

The document access system must wrap the document in either pdf.js or epub.js depending on format (is this possible in the theme? #67). The result must meet or exceed the WCAG 2.1 AA accessibility standard. The pdf.js and ePub.js libraries are believed to meet this accessibility requirement, but any additional elements required in order to use those libraries within uwka-pywb must be engineered to the same standard.

The system should also block Content-Dispostion: attachement downloads, and should display a contact/help page if the document in question is not HTML or PDF or ePub.

It is envisaged that support for other formats may be added in the future, either via client-side or server-side format conversion to HTML. This should be kept in mind when implementing this feature.

@ikreymer
Copy link
Contributor

Initial draft of support for pdf.js and epub.js is part of #85.

As mentioned in #74, content-disposition alone may not be enough to prevent downloads, if download prevention is needed for direct pywb access. If only loading from the viewer, there's a clear way to stop all downloads, which would be more fool-proof.

@anjackson
Copy link
Contributor Author

This works pretty well. Ideally, especially for printing, I'd like to experiment with some of ePub.js's other rendering and flow options. e.g. scrolled-doc or possibly even continuous-scrolled.

However, it looks to me like ePub Reader doesn't make it easy to change this. So maybe we should push this off to a future iteration.

@anjackson
Copy link
Contributor Author

As our original request has been implemented, I'm going to call this ticket done. We need some changes to the ePub view but we'll follow up those up under the follow-on support arrangement. See #100

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

No branches or pull requests

2 participants