Merge pull request #568 from biglotteryfund/image-cropping-changes

WEB-97 Updated default image crop
biglotteryfund · Mar 25, 2021 · 8195937 · 8195937
2 parents 21eae46 + 60a1489
commit 8195937
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/Images.php b/lib/Images.php
@@ -35,7 +35,7 @@ public static function imgixUrl($originalUrl, $options = [])
             $builder->setSignKey($imgixConfig['signKey']);
             $builder->setIncludeLibraryParam(false);
 
-            $defaults = array('auto' => "compress,format", 'crop' => 'entropy', 'fit' => 'crop');
+            $defaults = array('auto' => "compress,format", 'crop' => 'faces,edge', 'fit' => 'crop');
             $params = array_replace_recursive($defaults, $options);
             return $builder->createURL($parsedUri->getPath(), $params);
         } else {

diff --git a/tests/ImagesTest.php b/tests/ImagesTest.php
@@ -9,12 +9,12 @@ final class ImagesTest extends TestCase
     public function testBuildsImgixUrl(): void
     {
         $this->assertRegExp(
-            '/image\.jpg\?auto=compress%2Cformat&crop=entropy&fit=crop&s=.*$/',
+            '/image\.jpg\?auto=compress%2Cformat&crop=faces%2Cedge&fit=crop&s=.*$/',
             Images::imgixUrl("https://media.example.com/path/to/image.jpg")
         );
 
         $this->assertRegExp(
-            '/image\.jpg\?auto=compress%2Cformat&crop=entropy&fit=crop&w=100&s=.*$/',
+            '/image\.jpg\?auto=compress%2Cformat&crop=faces%2Cedge&fit=crop&w=100&s=.*$/',
             Images::imgixUrl("https://media.example.com/path/to/image.jpg", ['w' => 100])
         );
     }