Fixes #487 Made image zoomable after clicking on the image

Author: Ramkishorevit

app/build.gradle

@@ -13,7 +13,9 @@ apply plugin: 'io.fabric'
 repositories {
     maven { url 'https://maven.fabric.io/public' }
 }
-
+repositories {
+    maven { url "https://jitpack.io" }
+}
 
 android {
     compileSdkVersion 24
@@ -86,6 +88,7 @@ dependencies {
     compile 'com.github.bumptech.glide:glide:3.8.0'
     compile 'com.android.support:support-v4:24.1.0'
     compile 'com.theartofdev.edmodo:android-image-cropper:2.3.+'
+    compile 'com.github.chrisbanes:PhotoView:1.3.0'
     compile('com.crashlytics.sdk.android:crashlytics:2.6.7@aar') {
         transitive = true;
     }

app/src/main/java/com/zulip/android/activities/PhotoViewActivity.java

@@ -10,6 +10,8 @@
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.content.pm.PackageManager;
+import android.graphics.Bitmap;
+import android.graphics.drawable.Drawable;
 import android.net.ConnectivityManager;
 import android.net.NetworkInfo;
 import android.net.Uri;
@@ -29,15 +31,13 @@
 import android.widget.ImageView;
 import android.widget.ProgressBar;
 import android.widget.Toast;
-
 import com.bumptech.glide.Glide;
 import com.bumptech.glide.load.resource.drawable.GlideDrawable;
 import com.bumptech.glide.request.RequestListener;
-import com.bumptech.glide.request.target.GlideDrawableImageViewTarget;
-import com.bumptech.glide.request.target.Target;
+import com.bumptech.glide.request.animation.GlideAnimation;
+import com.bumptech.glide.request.target.SimpleTarget;
 import com.zulip.android.R;
 import com.zulip.android.util.ZLog;
-
 import java.io.BufferedInputStream;
 import java.io.File;
 import java.io.FileOutputStream;
@@ -47,6 +47,7 @@
 import java.net.MalformedURLException;
 import java.net.URL;
 import java.net.URLConnection;
+import uk.co.senab.photoview.PhotoView;
 
 
 public class PhotoViewActivity extends AppCompatActivity implements ActivityCompat.OnRequestPermissionsResultCallback {
@@ -55,7 +56,7 @@ public class PhotoViewActivity extends AppCompatActivity implements ActivityComp
     private static final int EXTERNAL_STORAGE_PERMISSION_CONSTANT = 100;
     private static final int REQUEST_PERMISSION_SETTING = 101;
     int id = 1;
-    private ImageView linkImage;
+    private PhotoView linkImage;
     private ProgressBar progressBar;
     private NotificationManager mNotifyManager;
     private NotificationCompat.Builder mBuilder;
@@ -74,31 +75,26 @@ protected void onCreate(Bundle savedInstanceState) {
         final Intent intent = getIntent();
         String url = intent.getStringExtra(Intent.EXTRA_TEXT);
         getSupportActionBar().setTitle(url);
-        linkImage = (ImageView) findViewById(R.id.linkImageView);
+        linkImage = (PhotoView) findViewById(R.id.linkImageView);
         progressBar = (ProgressBar) findViewById(R.id.progress);
-        GlideDrawableImageViewTarget imageViewPreview = new GlideDrawableImageViewTarget(linkImage);
-        Glide
-                .with(this)
-                .load(url)
-                .listener(new RequestListener<String, GlideDrawable>() {
-                    @Override
-                    public boolean onException(Exception e, String model, Target<GlideDrawable> target, boolean isFirstResource) {
-                        progressBar.setVisibility(View.GONE);
-                        if (!isNetworkAvailable()) {
-                            Toast.makeText(getApplicationContext(), R.string.toast_no_internet_connection, Toast.LENGTH_SHORT).show();
-                        } else {
-                            Toast.makeText(getApplicationContext(), R.string.toast_unable_to_load_image, Toast.LENGTH_SHORT).show();
-                        }
-                        return false;
-                    }
-
-                    @Override
-                    public boolean onResourceReady(GlideDrawable resource, String model, Target<GlideDrawable> target, boolean isFromMemoryCache, boolean isFirstResource) {
-                        progressBar.setVisibility(View.GONE);
-                        return false;
-                    }
-                })
-                .into(imageViewPreview);
+        Glide.with(this).load(url).asBitmap().into(new SimpleTarget<Bitmap>() {
+            @Override
+            public void onResourceReady(Bitmap resource, GlideAnimation<? super Bitmap> glideAnimation) {
+                progressBar.setVisibility(View.GONE);
+                linkImage.setImageBitmap(resource);
+            }
+
+            @Override
+            public void onLoadFailed(Exception e, Drawable errorDrawable) {
+                super.onLoadFailed(e, errorDrawable);
+                progressBar.setVisibility(View.GONE);
+                if (!isNetworkAvailable()) {
+                    Toast.makeText(getApplicationContext(), R.string.toast_no_internet_connection, Toast.LENGTH_SHORT).show();
+                } else {
+                    Toast.makeText(getApplicationContext(), R.string.toast_unable_to_load_image, Toast.LENGTH_SHORT).show();
+                }
+            }
+        });
     }
 
     private boolean isNetworkAvailable() {

app/src/main/res/layout/activity_photo_view.xml

@@ -20,7 +20,7 @@
         android:visibility="visible" />
 
 
-    <ImageView
+    <uk.co.senab.photoview.PhotoView
         android:id="@+id/linkImageView"
         android:layout_width="match_parent"
         android:layout_height="match_parent"