scientist-softserv / palni-palci

Other
1 stars 0 forks source link

Embed PDF viewer #125

Closed orangewolf closed 1 year ago

orangewolf commented 1 year ago

Summary

Implement pdf.js viewer: https://mozilla.github.io/pdf.js/

An embedded PDF viewer is an essential improvement for the implementation and use of Hyku by both our current and potential user base. They would like to have the option to use IIIF Print or the embedded PDF for PDF items through feature flippers.

Acceptance Criteria

Screenshots or Video

Example of non-cached pdf loading in JSTOR:

Image

Example of vertical format:

Image

Testing Instructions

Notes

Confused how to fill this out? Check out the playbook

DiemBTran commented 1 year ago

Linked items:

Relates to:

DiemBTran commented 1 year ago

Amanda - Oct 2020:


also make it able to turn off and on

ndroark commented 1 year ago

Here is a Hyrax repo with pdf.js installed https://ldr.lafayette.edu/concern/publications/9019s417m

kirkkwang commented 1 year ago

useful reference: https://github.com/LafayetteCollegeLibraries/spot/pull/14

jillpe commented 1 year ago

SoftServ QA:

PDF Link

Default Theme ✅ ![Image](https://github.com/scientist-softserv/palni-palci/assets/84697174/5892b01b-f9a2-4d04-8f8e-0b3ec0a19b6c)
Scholarly ✅ ![Image](https://github.com/scientist-softserv/palni-palci/assets/84697174/077c27e5-5268-4c44-9b10-8bbba8e7197c)
Culture ❌ Viewer is contained to left side of screen ![Image](https://github.com/scientist-softserv/palni-palci/assets/84697174/6565edf0-2278-4775-95e3-f796bafa5703)
ReShare ❌ The metadata is overlapping the viewer ![Image](https://github.com/scientist-softserv/palni-palci/assets/84697174/5a20eb73-72a8-4b32-9a7e-d22c867ddedd)
kirkkwang commented 1 year ago

ReShare will not be using PDF.js so no need to test with that theme.

Cultural theme viewer fixed (download button will be fixed in a separate ticket) Image