fix(image): image aspect dimensions for ImageSource.fromAsset(...) (NativeScript#5556)

manoldonev · web-flow · commit 7506905770ce · 2018-03-19T17:01:26.000+02:00
diff --git a/tests/app/image-source/image-source-tests.ts b/tests/app/image-source/image-source-tests.ts
@@ -141,8 +141,8 @@ export function testFromAssetWithScalingAndAspectRatio(done) {
     };
 
     let img = imageSource.fromAsset(asset).then((source) => {
-        TKUnit.assertEqual(source.width, scaleWidth);
-        TKUnit.assertEqual(source.height, 5);
+        TKUnit.assertEqual(source.width, 18);
+        TKUnit.assertEqual(source.height, scaleHeight);
         done();
     }, (error) => {
         done(error);
diff --git a/tns-core-modules/image-asset/image-asset-common.ts b/tns-core-modules/image-asset/image-asset-common.ts
@@ -33,8 +33,7 @@ export class ImageAsset  extends observable.Observable implements definition.Ima
 export function getAspectSafeDimensions(sourceWidth, sourceHeight, reqWidth, reqHeight) {
     let widthCoef = sourceWidth / reqWidth;
     let heightCoef = sourceHeight / reqHeight;
-
-    let aspectCoef = widthCoef > heightCoef ? widthCoef : heightCoef;
+    let aspectCoef = Math.min(widthCoef, heightCoef);
 
     return {
         width: Math.floor(sourceWidth / aspectCoef),
