@nativescript-community/ui-image 4.3.11 → 4.3.13

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.3.13](https://github.com/nativescript-community/ui-image/compare/v4.3.12...v4.3.13) (2023-09-27)
+
+### Bug Fixes
+
+* **android:** border radius correctly working now ([69c2732](https://github.com/nativescript-community/ui-image/commit/69c27327ebc627901a6aa4e7617b8975b3d14bc4))
+
+## [4.3.12](https://github.com/nativescript-community/ui-image/compare/v4.3.11...v4.3.12) (2023-09-26)
+
+### Bug Fixes
+
+* **android:** zoomimage not showing anything ([a8c666b](https://github.com/nativescript-community/ui-image/commit/a8c666bec8ffd0b3bc0907e0e9ddab89ec8dfbaf))
+
 ## [4.3.11](https://github.com/nativescript-community/ui-image/compare/v4.3.10...v4.3.11) (2023-08-15)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nativescript-community/ui-image",
-    "version": "4.3.11",
+    "version": "4.3.13",
     "description": "Advanced and efficient image display plugin which uses Fresco (Android) and SDWebImage (iOS) to implement caching, placeholders, image effects, and much more.",
     "main": "./index",
     "sideEffects": false,
@@ -44,5 +44,5 @@
     },
     "license": "Apache-2.0",
     "readmeFilename": "README.md",
-    "gitHead": "c4c455b78a420a512dd17bdc08b2aa5a0a0e4ed4"
+    "gitHead": "0191483b8075a1fc9d6f7c8ef2f6b7d8a0bd3a78"
 }

package/platforms/android/include.gradle

@@ -1,6 +1,6 @@
 import groovy.json.JsonSlurper
 dependencies {
-	def frescoVersion =  project.hasProperty("frescoVersion") ? project.frescoVersion : "3.0.0"
+	def frescoVersion =  project.hasProperty("frescoVersion") ? project.frescoVersion : "2.6.0"
 
 	implementation("com.facebook.fresco:fresco:$frescoVersion") {
         exclude group: 'com.facebook.soloader', module: 'soloader'
@@ -9,11 +9,7 @@ dependencies {
 	implementation("com.facebook.fresco:imagepipeline-okhttp3:$frescoVersion") {
 		exclude group: 'com.facebook.soloader', module: 'soloader'
 	}
-    implementation('com.facebook.fresco:nativeimagetranscoder') {
-        version {
-            strictly '2.6.0'
-        }
-    }
+    implementation("com.facebook.fresco:nativeimagetranscoder:$frescoVersion")
     implementation 'com.facebook.infer.annotation:infer-annotation:0.18.0'
 	// implementation ("com.facebook.fresco:animated-gif:$frescoVersion") {
 	// 	exclude group: 'com.facebook.soloader', module: 'soloader'

package/platforms/android/java/com/nativescript/image/DraweeView.java

@@ -4,6 +4,7 @@ import com.facebook.drawee.view.SimpleDraweeView;
 import com.facebook.imagepipeline.request.ImageRequestBuilder;
 import com.facebook.imagepipeline.request.ImageRequest;
 import com.facebook.drawee.backends.pipeline.PipelineDraweeControllerBuilder;
+import com.facebook.drawee.generic.GenericDraweeHierarchy;
 
 import android.graphics.Outline;
 import android.graphics.Rect;
@@ -13,6 +14,7 @@ import android.view.View;
 import android.content.Context;
 import android.os.Build;
 import android.util.Log;
+import android.util.AttributeSet;
 
 import android.graphics.Canvas;
 import android.graphics.Path;
@@ -34,58 +36,56 @@ import java.util.Arrays;
 public class DraweeView extends SimpleDraweeView {
     public int imageWidth = 0;
     public int imageHeight = 0;
+    public boolean isUsingOutlineProvider = false;
     private static Paint clipPaint;
 
-    private boolean clipEnabled = true;
-
-    public void setClipToBounds(boolean value) {
-        clipEnabled = value;
-        if (value) {
-            if (android.os.Build.VERSION.SDK_INT >= 21) {
-                setOutlineProvider(new ViewOutlineProvider() {
-                    @Override
-                    public void getOutline(View view, Outline outline) {
-                        Drawable drawable = getBackground();
-                        if (drawable instanceof BorderDrawable) {
-                            BorderDrawable borderDrawable = (BorderDrawable) drawable;
-                            float borderTopLeftRadius = borderDrawable.getBorderTopLeftRadius();
-                            float borderTopRightRadius = borderDrawable.getBorderTopRightRadius();
-                            float borderBottomRightRadius = borderDrawable.getBorderBottomRightRadius();
-                            float borderBottomLeftRadius = borderDrawable.getBorderBottomLeftRadius();
-                            float borderLeftWidth = borderDrawable.getBorderLeftWidth();
-                            float borderBottomWidth = borderDrawable.getBorderBottomWidth();
-                            float borderTopWidth = borderDrawable.getBorderTopWidth();
-                            float borderRightWidth = borderDrawable.getBorderRightWidth();
-                            if (android.os.Build.VERSION.SDK_INT >= 21 && 
-                                borderDrawable.getUniformBorderRadius() > 0 &&
-                                borderLeftWidth == 0 &&
-                                borderBottomWidth == 0 &&
-                                borderTopWidth == 0 &&
-                                borderRightWidth == 0) {
-                                drawable.getOutline(outline);
-                                return;
-                            }
+    public DraweeView(Context context, GenericDraweeHierarchy hierarchy) {
+        super(context);
+    }
+
+    public DraweeView(Context context) {
+        super(context);
+    }
+
+    public DraweeView(Context context, AttributeSet attrs) {
+        super(context, attrs);
+    }
+
+    public DraweeView(Context context, AttributeSet attrs, int defStyle) {
+        super(context, attrs, defStyle);
+    }
+
+	// private final Rect outlineRect = new RectF();
+    public void updateOutlineProvider() {
+        Drawable drawable = getBackground();
+        isUsingOutlineProvider = false;
+        if (android.os.Build.VERSION.SDK_INT >= 21 && drawable instanceof BorderDrawable && (android.os.Build.VERSION.SDK_INT >= 33 || ((BorderDrawable)drawable).hasUniformBorderRadius())) {
+            setOutlineProvider(new ViewOutlineProvider() {
+                @Override
+                public void getOutline(View view, Outline outline) {
+                    Drawable drawable = getBackground();
+                    if (drawable instanceof BorderDrawable) {
+                        BorderDrawable borderDrawable = (BorderDrawable) drawable;
+                        // that if test is only needed until N BorderDrawable is updated to do it
+                        if (borderDrawable.hasUniformBorderRadius()) {
+			                // outlineRect.set(borderDrawable.getBounds());
+                            outline.setRoundRect(borderDrawable.getBounds(), borderDrawable.getBorderBottomLeftRadius());
+                        } else {
+                            drawable.getOutline(outline);
                         }
-                        outline.setRect(0, 0, view.getWidth(), view.getHeight());
+                    } else {
+                        outline.setRect(100, 100, view.getWidth() - 200, view.getHeight() - 200);
                     }
-                });
-                setClipToOutline(true);
-            }
+                }
+            });
+            setClipToOutline(true);
+            isUsingOutlineProvider = true;
         } else if (android.os.Build.VERSION.SDK_INT >= 21) {
             setOutlineProvider(null);
             setClipToOutline(false);
         }
     }
 
-    public boolean getClipToBounds() {
-        return clipEnabled;
-    }
-
-    public DraweeView(Context context) {
-        super(context);
-        setClipToBounds(clipEnabled);
-    }
-
     @Override
     protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
         int widthMode = MeasureSpec.getMode(widthMeasureSpec);
@@ -123,13 +123,6 @@ public class DraweeView extends SimpleDraweeView {
         float borderBottomWidth = borderDrawable.getBorderBottomWidth();
         float borderTopWidth = borderDrawable.getBorderTopWidth();
         float borderRightWidth = borderDrawable.getBorderRightWidth();
-        if (android.os.Build.VERSION.SDK_INT >= 21 && ((borderDrawable.hasUniformBorderRadius()
-                && (borderLeftWidth == 0 && borderBottomWidth == 0 && borderTopWidth == 0 && borderRightWidth == 0))
-                || (borderTopLeftRadius == 0 && borderTopRightRadius == 0 && borderBottomRightRadius == 0
-                        && borderBottomLeftRadius == 0))) {
-                // we can use outline
-            return null;
-        }
         if (innerBorderPath == null) {
             innerBorderPath = new Path();
         } else {
@@ -162,7 +155,7 @@ public class DraweeView extends SimpleDraweeView {
     @Override
     protected void onDraw(Canvas canvas) {
         Drawable drawable = getBackground();
-        if (clipEnabled && drawable instanceof BorderDrawable) {
+        if (!isUsingOutlineProvider && drawable instanceof BorderDrawable) {
             BorderDrawable borderDrawable = (BorderDrawable) drawable;
             Path clipPath = generateInnerBorderPath(borderDrawable);
             if (clipPath != null) {
@@ -183,12 +176,11 @@ public class DraweeView extends SimpleDraweeView {
                 canvas.restoreToCount(saveCount);
                 return;
             }
-        }
+        } 
         super.onDraw(canvas);
     }
 
     public void setUri(android.net.Uri uri, String jsonOptions, com.facebook.drawee.controller.ControllerListener listener) {
-    long start = System.nanoTime();
     ImageRequestBuilder requestBuilder = ImageRequestBuilder.newBuilderWithSource(uri).setRotationOptions( com.facebook.imagepipeline.common.RotationOptions.autoRotate());
         JSONObject object = null;
         if (jsonOptions.length() > 2) {
@@ -249,7 +241,18 @@ public class DraweeView extends SimpleDraweeView {
                 builder.setTapToRetryEnabled(true);
             }
         }
-
         setController(builder.build());
     }
+
+    @Override
+    public void setBackground(Drawable background) {
+        super.setBackground(background);
+        updateOutlineProvider();
+    }
+
+    @Override
+    protected void onSizeChanged(int w, int h, int oldw, int oldh) {
+        super.onSizeChanged(w, h, oldw, oldh);
+          updateOutlineProvider();
+  }
 }