Fix reuse cell animation and now happens when the user clic the cell …

…and not when the user scroll and the reusable cell appears
rhalgravez · Jun 22, 2017 · 0f5c483 · 0f5c483
1 parent cff649a
commit 0f5c483
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 2 deletions.
diff --git a/CollectionSample/CollectionSample/AGTColorfulViewController.m b/CollectionSample/CollectionSample/AGTColorfulViewController.m
@@ -9,6 +9,7 @@
 #import "AGTColorfulViewController.h"
 #import "AGTColors.h"
 #import "AGTRandomColorCell.h"
+#import "UIColor+Colorful.h"
 
 @interface AGTColorfulViewController ()
 
@@ -58,7 +59,7 @@ -(instancetype)initWithModel:(AGTColors *)model layout:(UICollectionViewLayout *
     if (self = [super initWithCollectionViewLayout:layout]) {
         _model = model;
         self.title = @"United colors of Agbo";
-        self.maxRandomColorsToDisplay = 3;
+        self.maxRandomColorsToDisplay = 104;
     }
 
     return self;
@@ -170,4 +171,14 @@ -(UICollectionReusableView *)collectionView:(UICollectionView *)collectionView v
     return subView;
 }
 
+#pragma mark - Delegate
+-(void) collectionView:(UICollectionView *)collectionView didSelectItemAtIndexPath:(NSIndexPath *)indexPath{
+
+    if (indexPath.section == [AGTColorfulViewController radomColorSection]) {
+        AGTRandomColorCell *cell = (AGTRandomColorCell*)[collectionView cellForItemAtIndexPath:indexPath];
+        cell.color = [cell.color complementaryColor];
+    }
+
+}
+
 @end
diff --git a/CollectionSample/CollectionSample/AGTRandomColorCell.h b/CollectionSample/CollectionSample/AGTRandomColorCell.h
@@ -13,5 +13,6 @@
 @property (weak, nonatomic) IBOutlet UILabel *hexView;
 
 @property (nonatomic, strong) UIColor *color;
+@property (nonatomic) BOOL shouldAnimateChangeOfColor;
 
 @end
diff --git a/CollectionSample/CollectionSample/AGTRandomColorCell.m b/CollectionSample/CollectionSample/AGTRandomColorCell.m
@@ -12,13 +12,22 @@
 @implementation AGTRandomColorCell
 
 -(void)awakeFromNib {
+    self.shouldAnimateChangeOfColor = NO;
     [self setupKVO];
 }
 
 -(void) dealloc {
     [self tearDownKVO];
 }
 
+#pragma mark - View lifecycle
+-(void)prepareForReuse {
+    [super prepareForReuse];
+
+    self.shouldAnimateChangeOfColor = NO;
+    self.color = [UIColor x11WhiteColor];
+}
+
 #pragma mark - KVO
 -(void)setupKVO {
     [self addObserver:self forKeyPath:@"color" options:NSKeyValueObservingOptionNew | NSKeyValueObservingOptionOld context:NULL];
@@ -34,10 +43,17 @@ -(void)observeValueForKeyPath:(NSString *)keyPath
                       context:(void *)context {
     self.hexView.text = [self.color hexString];
 
-    [UIView animateWithDuration:0.6 animations:^{
+    float  duration = 0.0f;
+    if (self.shouldAnimateChangeOfColor) {
+        duration = 0.6f;
+    }
+
+    [UIView animateWithDuration:duration animations:^{
         self.backgroundColor = self.color;
         self.hexView.textColor = [self.color contrastingTextColor];
     }];
+
+    self.shouldAnimateChangeOfColor = YES;
 }
 
 @end