getPreviewIfAny: Use physical pixel size for preview resolution

Code was inspired by code in an MDN page (in the public domain) https://developer.mozilla.org/en-US/docs/Web/API/Window/devicePixelRatio getPreviewIfAny: extract queryParams to get preview Signed-off-by: Robbert Gurdeep Singh <[email protected]> Signed-off-by: Axel Pirek <[email protected]> Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
nextcloud · Jun 5, 2021 · 49b70c6 · 49b70c6
1 parent 0ea0add
commit 49b70c6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/js/viewer-main.js b/js/viewer-main.js
diff --git a/js/viewer-main.js.map b/js/viewer-main.js.map
diff --git a/src/mixins/PreviewUrl.js b/src/mixins/PreviewUrl.js
@@ -63,12 +63,17 @@ export default {
 		 * @returns {String} the absolute url
 		 */
 		getPreviewIfAny({ fileid, filename, hasPreview, davPath }) {
+			const searchParams = `fileId=${fileid}`
+				+ `&x=${Math.floor(screen.width * devicePixelRatio)}`
+				+ `&y=${Math.floor(screen.height * devicePixelRatio)}`
+				+ '&a=true'
+
 			if (hasPreview) {
 				// TODO: find a nicer standard way of doing this?
 				if (isPublic()) {
-					return generateUrl(`/apps/files_sharing/publicpreview/${getToken()}?fileId=${fileid}&file=${encodeFilePath(filename)}&x=${screen.width}&y=${screen.height}&a=true`)
+					return generateUrl(`/apps/files_sharing/publicpreview/${getToken()}?file=${encodeFilePath(filename)}&${searchParams}`)
 				}
-				return generateUrl(`/core/preview?fileId=${fileid}&x=${screen.width}&y=${screen.height}&a=true`)
+				return generateUrl(`/core/preview?${searchParams}`)
 			}
 			return davPath
 		},