diff options
author | Adam Copp <adamcopp@google.com> | 2012-09-11 17:21:36 +0100 |
---|---|---|
committer | Adam Copp <adamcopp@google.com> | 2012-09-12 12:20:04 +0100 |
commit | 04e714257e8d6da434cb2d23feab365d38d05273 (patch) | |
tree | b38abf1e855eddc7c7ce5faee283baf356aa9dad /photoviewer | |
parent | 024a256f40a302ca927881bf43a689a59acef28e (diff) | |
download | android_frameworks_ex-04e714257e8d6da434cb2d23feab365d38d05273.tar.gz android_frameworks_ex-04e714257e8d6da434cb2d23feab365d38d05273.tar.bz2 android_frameworks_ex-04e714257e8d6da434cb2d23feab365d38d05273.zip |
Do not show loading bar after thumbnail has loaded
After talking to the designers for Google Now, who have said they talked
to the designers from gallery app, it appears that we should not show
the loading spinner once the thumbnail has loaded.
Let me know if this is orthogonal to the behaviour you would like in the
gmail app, and I can write a more involved patch with an overridable
callback to decide wether to show the spinner or not.
Change-Id: I77a3f2e2487d469ab47e45c7a94942f831528d41
Diffstat (limited to 'photoviewer')
-rw-r--r-- | photoviewer/src/com/android/ex/photo/fragments/PhotoViewFragment.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/photoviewer/src/com/android/ex/photo/fragments/PhotoViewFragment.java b/photoviewer/src/com/android/ex/photo/fragments/PhotoViewFragment.java index 7be37b0..c6c2e3b 100644 --- a/photoviewer/src/com/android/ex/photo/fragments/PhotoViewFragment.java +++ b/photoviewer/src/com/android/ex/photo/fragments/PhotoViewFragment.java @@ -272,18 +272,16 @@ public class PhotoViewFragment extends Fragment implements } break; case LOADER_ID_THUMBNAIL: + mProgressBarNeeded = false; if (isPhotoBound()) { // There is need to do anything with the thumbnail image, as the full size // image is being shown. mPhotoPreviewAndProgress.setVisibility(View.GONE); - mProgressBarNeeded = false; return; } else if (data == null) { // no preview, show default mPhotoPreviewImage.setVisibility(View.VISIBLE); mPhotoPreviewImage.setImageResource(R.drawable.default_image); - - mProgressBarNeeded = false; } else { bindPhoto(data); getLoaderManager().initLoader(LOADER_ID_PHOTO, null, this); |