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
feat(resource): add document viewer with download (DSP-1791) #485
Conversation
same view as for documents
…ch-swiss/dsp-app into wip/dsp-1791-document-viewer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me open file in new tab
button is not working but it is expected as I am using the test server for dsp-api. Other than that everything looks fine to me.
I will let @mdelez to approve the PR as he could also test the open file in new tab
functionality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works great and I can open my files in a new tab :)
resolves DSP-1791
In this PR I actually wanted to integrate a simple download button for pdf file representations. But the main work was to generalise how we handle the different representation types. Previous everything was prepared for still images only. This is why I had to create a general
FileRepresentation
type. Additional I integrated a simple angular pdf viewer (ng2-pdf-viewer) and styled it the same way as we already did for the opeansedragon viewer for still images. I just had to adjust the still image viewer resp. I had to move the style to our main styling files to use the same setup for the new document viewer.