fixed isLoading to show indicator

Expensify · Mar 1, 2023 · 8304865 · 8304865
1 parent 6bfc8bb
commit 8304865
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/components/ImageView/index.native.js b/src/components/ImageView/index.native.js
@@ -56,13 +56,13 @@ class ImageView extends PureComponent {
         });
 
         this.configureImageZoom = this.configureImageZoom.bind(this);
+        this.imageLoadingStart = this.imageLoadingStart.bind(this);
     }
 
     componentDidUpdate(prevProps) {
         if (this.props.url === prevProps.url || !this.interactionPromise) {
             return;
         }
-        this.imageLoadStart();
         this.state.interactionPromise.cancel();
     }
 
@@ -99,6 +99,7 @@ class ImageView extends PureComponent {
     configureImageZoom({nativeEvent}) {
         // Wait till animations are over to prevent stutter in navigation animation
         this.state.interactionPromise = InteractionManager.runAfterInteractions(() => {
+            this.setState({ imageHeight: 0, imageWidth: 0, isLoading: true });
             if (this.imageZoomScale !== 1) {
                 this.imageZoomScale = 1;
             }
@@ -133,6 +134,10 @@ class ImageView extends PureComponent {
         });
     }
 
+    imageLoadingStart() {
+        this.setState({ isLoading: true });
+    }
+
     render() {
         // Default windowHeight accounts for the modal header height
         const windowHeight = this.props.windowHeight - variables.contentHeaderHeight;
@@ -205,6 +210,7 @@ class ImageView extends PureComponent {
                             source={{uri: this.props.url}}
                             isAuthTokenRequired={this.props.isAuthTokenRequired}
                             resizeMode={Image.resizeMode.contain}
+                            onLoadStart={this.imageLoadingStart}
                             onLoad={this.configureImageZoom}
                         />
                         {/**