Merge branch 'reduce_thumbnails' into thumbnail_wrong_filetype

lrsjng · Jan 23, 2021 · b7559ff · b7559ff
2 parents da82d88 + 3756441
commit b7559ff
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/src/_h5ai/private/conf/options.json b/src/_h5ai/private/conf/options.json
@@ -355,7 +355,7 @@
     - mov: array of strings
     - doc: array of strings
     - delay: number, delay in milliseconds after "dom-ready" before thumb-requesting starts
-    - size: number, size in pixel of the generated thumbnails
+    - size: number, height in pixel of the generated thumbnails
     - seek: number, percentage of total video duration to seek into
     - exif: boolean, use included EXIF thumbs if possible
     - chunksize: int, number of thumbs per request

diff --git a/src/_h5ai/private/php/core/class-context.php b/src/_h5ai/private/php/core/class-context.php
@@ -17,6 +17,9 @@ public function __construct($session, $request, $setup) {
 
         $this->options = Json::load($this->setup->get('CONF_PATH') . '/options.json');
 
+        $this->thumbnail_height = $this->options['thumbnails']['size'] ?? 240;
+        $this->thumbnail_width = floor($this->thumbnail_height * (4 / 3));
+
         $this->passhash = $this->query_option('passhash', '');
         $this->options['hasCustomPasshash'] = strcasecmp($this->passhash, Context::$DEFAULT_PASSHASH) !== 0;
         unset($this->options['passhash']);
@@ -248,7 +251,7 @@ public function get_thumbs($requests) {
                 $hrefs[] = null;
                 continue;
             }
-            $hrefs[] = $thumbs[$href]->thumb($req['width'], $req['height']);
+            $hrefs[] = $thumbs[$href]->thumb($this->thumbnail_width, $this->thumbnail_height);
         }
         return $hrefs;
     }

diff --git a/src/_h5ai/public/css/lib/view/view.less b/src/_h5ai/public/css/lib/view/view.less
@@ -48,6 +48,8 @@
         .thumb {
             max-width: none;
             max-height: none;
+            object-fit: cover;
+            object-position: 50% 50%;
         }
     }
 

diff --git a/src/_h5ai/public/js/lib/ext/thumbnails.js b/src/_h5ai/public/js/lib/ext/thumbnails.js
@@ -13,7 +13,6 @@ const settings = Object.assign({
     exif: false,
     chunksize: 20
 }, allsettings.thumbnails);
-const landscapeRatio = 4 / 3;
 
 
 const queueItem = (queue, item) => {
@@ -37,7 +36,6 @@ const queueItem = (queue, item) => {
         queue.push({
             type,
             href: item.absHref,
-            ratio: 1,
             callback: src => {
                 if (src && item.$view) {
                     item.thumbSquare = src;
@@ -53,7 +51,6 @@ const queueItem = (queue, item) => {
         queue.push({
             type,
             href: item.absHref,
-            ratio: landscapeRatio,
             callback: src => {
                 if (src && item.$view) {
                     item.thumbRational = src;
@@ -69,8 +66,6 @@ const requestQueue = queue => {
         return {
             type: req.type,
             href: req.href,
-            width: Math.round(settings.size * req.ratio),
-            height: settings.size
         };
     });