Use nextcloud/viewer

[szaimen]

Please enable us to open pdf files with the viewer app. Would be really awesome beeing able to open pdf files also in other apps like e.g. nextcloud mail for attachments. :)
This is possible since nextcloud/viewer#271

[skjnldsv]

I guess it make sense for pdf since it's read only yes :)

[skjnldsv]

Though It should probably go into the files_pdfviewer app

[juliusknorr]

I've heard that @danxuliu has been trying to move files_pdfviewer to webpack and update pdfjs so once that has some usabale state it could make use of viewer to integrate in the file list.

[szaimen]

@danxuliu is there any update concerning this? :)

[danxuliu]

It is high in my TODO, but unfortunately there have been other things with a higher priority on top ;-) But hopefully I will find some time to work on it soon™

[skjnldsv]

Done

[szaimen]

Wow that was fast 🎉

[skjnldsv]

I broke my brain for two days trying to implement everything without an iframe and proper library, it's not feasible.

So I went back, and implemented the iframe frontend from scratch. Now it's at least super clean and faster :)

[onny]

Wow that sounds impressive, could you share an screenshot? :O :D

[skjnldsv]

I'm out for the day :)
It will be part of nc20
It's basically just integrated into the Viewer that displays all the images, videos... Etc 🥳

[onny]

With a few css styling fixes one could get rid of the dysfunctional close button and the background pattern in pdfjs 👍