Modify image attachments previews to conform to UX specs

Thumbnail and full image start loading at the same time.

Once thumbnail is shown, we display a determinate progress bar while downloading
the full image.

Remove progress bar from image tiles in ConversationView.

Bug: 8021747
Change-Id: I600a8bb8337dd7ee6e7cec2a1221824b04a9e2ef

1 files changed, 1 insertions, 11 deletions

diff --git a/src/com/android/mail/browse/MessageAttachmentTile.java b/src/com/android/mail/browse/MessageAttachmentTile.java
index d6f19c31d..d88aa8152 100644
--- a/src/com/android/mail/browse/MessageAttachmentTile.java
+++ b/src/com/android/mail/browse/MessageAttachmentTile.java
@@ -28,7 +28,6 @@ import android.view.LayoutInflater;
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.ViewGroup;
-import android.widget.ProgressBar;
 
 import com.android.ex.photo.Intents;
 import com.android.ex.photo.Intents.PhotoViewIntentBuilder;
@@ -60,7 +59,6 @@ public class MessageAttachmentTile extends AttachmentTile implements OnClickList
     private View mTextContainer;
 
     private final AttachmentActionHandler mActionHandler;
-    private ProgressBar mProgress;
 
     private static final String LOG_TAG = LogTag.getLogTag();
 
@@ -107,7 +105,6 @@ public class MessageAttachmentTile extends AttachmentTile implements OnClickList
         super.onFinishInflate();
 
         mTextContainer = findViewById(R.id.attachment_tile_text_container);
-        mProgress = (ProgressBar) findViewById(R.id.attachment_progress);
 
         setOnClickListener(this);
     }
@@ -152,14 +149,7 @@ public class MessageAttachmentTile extends AttachmentTile implements OnClickList
 
     @Override
     public void updateProgress(boolean showDeterminateProgress) {
-        if (mAttachment.isDownloading()) {
-            mProgress.setMax(mAttachment.size);
-            mProgress.setProgress(mAttachment.downloadedSize);
-            mProgress.setIndeterminate(!showDeterminateProgress);
-            mProgress.setVisibility(VISIBLE);
-        } else {
-            mProgress.setVisibility(GONE);
-        }
+        // do not show progress for image tiles
     }
 
     @Override