@nativescript-community/ui-image 4.3.15 → 4.3.17

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.3.17](https://github.com/nativescript-community/ui-image/compare/v4.3.16...v4.3.17) (2023-10-05)
+
+**Note:** Version bump only for package @nativescript-community/ui-image
+
+## [4.3.16](https://github.com/nativescript-community/ui-image/compare/v4.3.15...v4.3.16) (2023-09-29)
+
+### Bug Fixes
+
+* **android:** fix for image disappearing on fragment/ativity transitions. See https://github.com/facebook/fresco/issues/1446 ([b36d001](https://github.com/nativescript-community/ui-image/commit/b36d0012af443fadcdb969e901f7f0e69f4e0c80))
+
 ## [4.3.15](https://github.com/nativescript-community/ui-image/compare/v4.3.14...v4.3.15) (2023-09-28)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nativescript-community/ui-image",
-    "version": "4.3.15",
+    "version": "4.3.17",
     "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": "b2f5267f7fb2098d3bc7d6c77f4c88f6235005b7"
+    "gitHead": "8b04fc76e2cf7a61516801c2aae0739b0d394428"
 }

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

@@ -39,12 +39,24 @@ public class DraweeView extends SimpleDraweeView {
     public boolean isUsingOutlineProvider = false;
     private static Paint clipPaint;
 
+    private boolean mLegacyVisibilityHandlingEnabled = false;
+
+    private static boolean sGlobalLegacyVisibilityHandlingEnabled = true;
+
+    public static void setGlobalLegacyVisibilityHandlingEnabled(
+        boolean legacyVisibilityHandlingEnabled) {
+        sGlobalLegacyVisibilityHandlingEnabled = legacyVisibilityHandlingEnabled;
+    }
+
     public DraweeView(Context context, GenericDraweeHierarchy hierarchy) {
         super(context);
     }
 
     public DraweeView(Context context) {
         super(context);
+        // In Android N and above, visibility handling for Drawables has been changed, which breaks
+        // activity transitions with DraweeViews.
+        mLegacyVisibilityHandlingEnabled = sGlobalLegacyVisibilityHandlingEnabled  && context.getApplicationInfo().targetSdkVersion >= 24; //Build.VERSION_CODES.N
     }
 
     public DraweeView(Context context, AttributeSet attrs) {
@@ -55,6 +67,61 @@ public class DraweeView extends SimpleDraweeView {
         super(context, attrs, defStyle);
     }
 
+    @Override
+    protected void onAttachedToWindow() {
+        super.onAttachedToWindow();
+        maybeOverrideVisibilityHandling();
+        onAttach();
+    }
+
+    @Override
+    protected void onDetachedFromWindow() {
+        super.onDetachedFromWindow();
+        maybeOverrideVisibilityHandling();
+        onDetach();
+    }
+
+    @Override
+    public void onStartTemporaryDetach() {
+        super.onStartTemporaryDetach();
+        maybeOverrideVisibilityHandling();
+        onDetach();
+    }
+
+    @Override
+    public void onFinishTemporaryDetach() {
+        super.onFinishTemporaryDetach();
+        maybeOverrideVisibilityHandling();
+        onAttach();
+    }
+
+    @Override
+    protected void onVisibilityChanged(
+        View changedView,
+        int visibility) {
+        super.onVisibilityChanged(changedView, visibility);
+        maybeOverrideVisibilityHandling();
+    }
+
+    @Override
+    public void onVisibilityAggregated (boolean isVisible) {
+        super.onVisibilityAggregated(isVisible);
+        maybeOverrideVisibilityHandling();
+    }
+
+    private void maybeOverrideVisibilityHandling() {
+        if (mLegacyVisibilityHandlingEnabled)  {
+            Drawable drawable = getDrawable();
+            if (drawable != null) {
+                drawable.setVisible(getVisibility() == VISIBLE, false);
+            }
+        }
+    }
+
+    public void setLegacyVisibilityHandlingEnabled(boolean legacyVisibilityHandlingEnabled) {
+        mLegacyVisibilityHandlingEnabled = legacyVisibilityHandlingEnabled;
+    }
+
 	// private final Rect outlineRect = new RectF();
     public void updateOutlineProvider() {
         Drawable drawable = getBackground();

package/angular/directives.d.ts

@@ -1,7 +0,0 @@
-import * as i0 from "@angular/core";
-export declare class ImgDirective {
-    constructor();
-    static ɵfac: i0.ɵɵFactoryDeclaration<ImgDirective, never>;
-    static ɵdir: i0.ɵɵDirectiveDeclaration<ImgDirective, "NSImg", never, {}, {}, never, never, false, never>;
-}
-export declare const NSIMG_DIRECTIVES: (typeof ImgDirective)[];

package/angular/index.d.ts

@@ -1 +0,0 @@
-export * from './module';

package/angular/module.d.ts

@@ -1,9 +0,0 @@
-import { ImgDirective } from './directives';
-import * as i0 from "@angular/core";
-import * as i1 from "./directives";
-export { ImgDirective };
-export declare class TNSImageModule {
-    static ɵfac: i0.ɵɵFactoryDeclaration<TNSImageModule, never>;
-    static ɵmod: i0.ɵɵNgModuleDeclaration<TNSImageModule, [typeof i1.ImgDirective], never, [typeof i1.ImgDirective]>;
-    static ɵinj: i0.ɵɵInjectorDeclaration<TNSImageModule>;
-}