Corrected use of holder position.

canojim · Nov 24, 2015 · b969b50 · b969b50
1 parent 857c311
commit b969b50
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/app/src/main/java/io/plaidapp/ui/FeedAdapter.java b/app/src/main/java/io/plaidapp/ui/FeedAdapter.java
@@ -146,7 +146,7 @@ && getDataItemCount() > 0) {
             if (item instanceof Story) {
                 bindDesignerNewsStory((Story) getItem(position), (DesignerNewsStoryHolder) holder);
             } else if (item instanceof Shot) {
-                bindDribbbleShotView((Shot) item, (DribbbleShotHolder) holder, position);
+                bindDribbbleShotView((Shot) item, (DribbbleShotHolder) holder);
             } else if (item instanceof Post) {
                 bindProductHuntPostView((Post) item, (ProductHuntStoryHolder) holder);
             }
@@ -278,8 +278,7 @@ public void onAnimationEnd(Animator animation) {
     }
 
     private void bindDribbbleShotView(final Shot shot,
-                                      final DribbbleShotHolder holder,
-                                      final int position) {
+                                      final DribbbleShotHolder holder) {
         final BadgedFourThreeImageView iv = (BadgedFourThreeImageView) holder.itemView;
         final int[] imageSize = shot.images.bestSize();
         Glide.with(host)
@@ -330,7 +329,8 @@ public boolean onException(Exception e, String model, Target<GlideDrawable>
                         return false;
                     }
                 })
-                .placeholder(shotLoadingPlaceholders[position % shotLoadingPlaceholders.length])
+                .placeholder(shotLoadingPlaceholders[holder.getAdapterPosition() %
+                        shotLoadingPlaceholders.length])
                 .diskCacheStrategy(DiskCacheStrategy.SOURCE)
                 .fitCenter()
                 .override(imageSize[0], imageSize[1])