Changeset 28219

Timestamp:

04/25/2014 08:24:28 PM (11 years ago)

Author:

ocean90

Message:

Customizer: Allow to skip cropping header images if image width is smaller than or equal to theme width.

Add tests.

see #27936 for trunk.

Location:

trunk

Files:

• src/wp-includes/js/customize-models.js (modified) (1 diff)
• tests/qunit/wp-admin/js/customize-header.js (modified) (1 diff)

trunk/src/wp-includes/js/customize-models.js

@@ -114 +114 @@
             }
 
+            if (this.get('imageWidth') <= this.get('themeWidth')) {
+                return false;
+            }
+
             return true;
         }

trunk/tests/qunit/wp-admin/js/customize-header.js

@@ -104 +104 @@
             imageHeight: 8600
         });
+
+        equal(this.model.shouldBeCropped(), false);
+    });
+
+    test('should not be cropped when the image width is smaller than or equal to theme width', function() {
+        this.model.set({
+            themeFlexWidth: false,
+            themeFlexHeight: false,
+            imageWidth: 1000,
+            imageHeight: 100
+        });
+
+        equal(this.model.shouldBeCropped(), false);
+    });
+
+    test('should not be cropped when the image width is smaller than or equal to theme width, theme supports flex height and width', function() {
+        this.model.set({
+            themeFlexWidth: true,
+            themeFlexHeight: true,
+            imageWidth: 900,
+            imageHeight: 100
+        });
+
         equal(this.model.shouldBeCropped(), false);
     });