#25464: Applied code review suggestions

Author: sivaschenko

app/code/Magento/Ui/view/base/web/js/grid/columns/image-preview.js

@@ -28,6 +28,9 @@ define([
             listens: {
                 '${ $.provider }:params.filters': 'hide',
                 '${ $.provider }:params.search': 'hide'
+            },
+            exports: {
+                height: '${ $.parentName }.thumbnail_url:previewHeight'
             }
         },
 
@@ -43,9 +46,6 @@ define([
                     'displayedRecord',
                     'lastOpenedImage'
                 ]);
-            this.height.subscribe(function () {
-                this.thumbnailComponent().previewHeight(this.height());
-            }, this);
 
             return this;
         },

app/code/Magento/Ui/view/base/web/js/grid/masonry.js

@@ -48,7 +48,26 @@ define([
              * Maximum image height value
              * @param int
              */
-            maxImageHeight: 240
+            maxImageHeight: 240,
+
+            /**
+             * The value is minimum image width to height ratio when container width is less than the key
+             *
+             * @param int
+             */
+            containerWidthToMinRatio: {
+                640: 3,
+                1280: 5,
+                1920: 8
+            },
+
+            /**
+             * Default minimal image width to height ratio.
+             * Applied when container width is greater than max width in the containerWidthToMinRatio matrix.
+             *
+             * @param int
+             */
+            defaultMinRatio: 10
         },
 
         /**
@@ -229,15 +248,17 @@ define([
          * Set min ratio for images in layout
          */
         setMinRatio: function () {
-            if (this.containerWidth <= 640) {
-                this.minRatio = 3;
-            } else if (this.containerWidth <= 1280) {
-                this.minRatio = 5;
-            } else if (this.containerWidth <= 1920) {
-                this.minRatio = 8;
-            } else {
-                this.minRatio = 10;
+            var minRatio = null;
+
+            for (var width in this.containerWidthToMinRatio) {
+                if (this.containerWidthToMinRatio.hasOwnProperty(width) &&
+                    this.containerWidth <= width
+                ) {
+                    minRatio = this.containerWidthToMinRatio[width]
+                }
             }
+
+            this.minRatio = minRatio ? minRatio : this.defaultMinRatio;
         },
 
         /**