Age | Commit message (Collapse) | Author |
|
Bumps [pdfjs-dist](https://github.com/mozilla/pdfjs-dist) from 2.12.313 to 2.13.216.
- [Release notes](https://github.com/mozilla/pdfjs-dist/releases)
- [Commits](https://github.com/mozilla/pdfjs-dist/compare/v2.12.313...v2.13.216)
---
updated-dependencies:
- dependency-name: pdfjs-dist
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
|
|
Bumps [pdfjs-dist](https://github.com/mozilla/pdfjs-dist) from 2.10.377 to 2.12.313.
- [Release notes](https://github.com/mozilla/pdfjs-dist/releases)
- [Commits](https://github.com/mozilla/pdfjs-dist/commits)
---
updated-dependencies:
- dependency-name: pdfjs-dist
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
|
|
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
|
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
|
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
|
|
|
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
|
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
|
The "viewer.php" acts as the "viewer.html" file included in the PDF.js
package. This commit updates "viewer.php" with the changes to
"viewer.html" from PDF.js 1.9.426 to 2.0.943 while respecting the
customized parts of "viewer.php" (like the close button or the "nonce"
attribute).
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
The character maps are used by PDF.JS for font rendering, and are needed
to properly show some texts (for example, Japanese text). The PDF.JS
package provides the needed character maps, but the URL to get them must
be set with the "PDFJS.cMapUrl" property in the same way as the
"PDFJs.workerSrc" property is set (as the default value set in
"viewer.js" is not right for the PDF viewer app).
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
The scripts "compatibility.js" and "l10n.js" were removed when PDF.js
was updated to 1.9.426.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
"getAppVersion()" expects the ID of the app, but "$app" is not defined
in the templates, so it was just replaced by the hardcoded ID of the
app.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
|
|
Due to the changes from PDF.js 1.4.20 to 1.9.426 now trying to open a
different PDF file fails with a Content Security Policy violation.
Instead of fixing that the "Open file" button was disabled, as being
able to open a different PDF file from the local file system to view it
instead of the PDF file in the server did not provide much value.
The button was disabled by setting the "hidden" attribute to "true"
instead of with the CSS class for consistency with how it is done in
"vendor/pdfjs/web/viewer.js" when the browser does not support any of
the APIs needed to open the local file.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
The "viewer.php" acts as the "viewer.html" file included in the PDF.js
package. This commit updates "viewer.php" with the changes to
"viewer.html" from PDF.js 1.4.20 to 1.9.426 while respecting the
customized parts of "viewer.php" (like the close button or the "nonce"
attribute).
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
|
|
Signed-off-by: Robin Appelman <robin@icewind.nl>
|
|
|
|
Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
|
|
|
|
Add back close button
|
|
|
|
|
|
|
|
|
|
new file: img/toolbarButton-secondaryToolbarClose.png
new file: img/toolbarButton-secondaryToolbarClose@2x.png
modified: js/previewplugin.js
modified: templates/viewer.php
|
|
With ownCloud 8 the "Close" button of the PDF Viewer is shown incorrectly and overlaps the "secondary tool bar" button which results in a bad user experience and looks very ugly.
This patch addresses this problem by moving the "Close" button inside the iframe and accessing it using jQuery.
Testplan:
- [ ] Accessing a PDF as logged-in user from the file list works and a Close button is shown.
- [ ] Clicking on the close button closes the file
- [ ] Sharing a single PDF file via link works and no Close button is shown
- [ ] Sharing a folder containing a PDF file via link works and a Close button is shown.
- [ ] Clicking on the close button closes the file
To test this properly you need probably to checkout https://github.com/owncloud/core/pull/14773
|
|
Otherwise when updating ownCloud and the JS files have changed some loading issues may appear.
|
|
|
|
This is an ugly hack but ok for now. Let's fix this with a refactoring later.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
PDF viewer now correctly detects the files app.
Added missing JS dependency.
Fixes #1409
|
|
|
|
|
|
|
|
|
|
|
|
Pretty much based on viewer.html, bumping this as well.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|