heightForCellWithIdentifier -> heightForResizableCellWithIdentifier and support UITextView

Author: Ernesto Rivera

Demo/TableDemo/PEPhotosDetailViewController.m

@@ -167,9 +167,9 @@ - (AMBTableViewSection *)commentsSection
                                 }
 
                                 // Dynamic height comments
-                                return [weakSelf heightForCellWithIdentifier:@"comment"
-                                                                        text:object
-                                                      limitedToNumberOfLines:0];
+                                return [weakSelf heightForResizableCellWithIdentifier:@"comment"
+                                                                                 text:object
+                                                               limitedToNumberOfLines:0];
                             }
                             cellIdentifierBlock:^NSString *(id object,
                                                             NSIndexPath * indexPath)
@@ -193,7 +193,7 @@ - (AMBTableViewSection *)commentsSection
         {
             return @"Comments";
         };
-
+        
         // Enable "no content cell"
         _commentsSection.presentsNoContentCell = YES;
     }
@@ -336,7 +336,7 @@ @implementation PEPhotosDetailAuthorCell
 
 @implementation PEPhotosDetailCommentCell
 
-@synthesize resizableLabel = _resizableLabel;
+@synthesize resizableView = _resizableView;
 
 @end
 

Source/AMBTableViewController.h

@@ -110,16 +110,18 @@
 /// @param subview A subview of the row.
 - (NSIndexPath *)indexPathForRowWithSubview:(UIView *)subview;
 
-/// Calculate the required height for a dynamic height label.
+/// Calculate the required height for a AMBResizableCell.
 /// @param identifier The identifier of the AMBResizableCell who's height will be calculated.
-/// @param text The text that the [AMBResizableCell resizableLabel] should fit.
-/// @param numberOfLines The maximum number of lines to be used by the label or `0` for no limits.
+/// @param text The text that the view should fit.
+/// @param numberOfLines The maximum number of lines to be allowed or `0` for no limits.
+/// Only implemented for UILabel resizableView's.
 /// @discussion A cell instance of the given identifier will be cached to calculate the heights
 /// of all future calculations. The original height of the loaded cell will be used as the
 /// minimum height to be returned by the function.
-- (CGFloat)heightForCellWithIdentifier:(NSString *)identifier
-                                  text:(NSString *)text
-                limitedToNumberOfLines:(NSInteger)numberOfLines;
+/// @note Currently only suport for UILabel and UITextView objects is implemented.
+- (CGFloat)heightForResizableCellWithIdentifier:(NSString *)identifier
+                                           text:(NSString *)text
+                         limitedToNumberOfLines:(NSInteger)numberOfLines;
 
 @end
 
@@ -397,17 +399,18 @@ typedef void (^AMBTableViewCellConfigurationBlock)(id object,
 
 /**
  A protocol that when implemented by a UITableViewCell enables using
- [AMBTableViewController heightForCellWithIdentifier:text:limitedToNumberOfLines:]
+ [AMBTableViewController heightForResizableCellWithIdentifier:text:limitedToNumberOfLines:]
  to easily calculate dynamic-sized cells.
+ @note Currently only suport for UILabel and UITextView objects is implemented.
  */
 @protocol AMBResizableCell <NSObject>
 
 /// The label that determines the required height of the cell.
-/// @discussion The original cell height is saved and used as the minimum height to used
-/// for the cell.
-/// @note The label should be set to use flexible height and automatically grow/shrink
-/// when its parent cell changes adjusts its height.
-@property (weak, nonatomic) IBOutlet UILabel * resizableLabel;
+/// @discussion The original cell height is saved and used as the minimum height
+/// to be used for the cell.
+/// @note The label should be set to use flexible height and automatically
+/// grow/shrink when its parent cell adjusts its height.
+@property (weak, nonatomic) IBOutlet UIView * resizableView;
 
 @end
 

Source/AMBTableViewController.m

@@ -197,9 +197,9 @@ - (NSIndexPath *)indexPathForRowWithSubview:(UIView *)subview
     return [self.tableView indexPathForRowAtPoint:point];
 }
 
-- (CGFloat)heightForCellWithIdentifier:(NSString *)identifier
-                                  text:(NSString *)text
-                limitedToNumberOfLines:(NSInteger)numberOfLines
+- (CGFloat)heightForResizableCellWithIdentifier:(NSString *)identifier
+                                           text:(NSString *)text
+                         limitedToNumberOfLines:(NSInteger)numberOfLines
 {
     static NSMutableDictionary * cache;
     static dispatch_once_t onceToken;
@@ -212,7 +212,7 @@ - (CGFloat)heightForCellWithIdentifier:(NSString *)identifier
     NSDictionary * cachedValues = cache[uniqueCellIdenfier];
     CGFloat minimumHeight;
     CGSize sizeDifference;
-    UILabel * label;
+    UIView * resizableView;
 
     if (!cachedValues)
     {
@@ -221,35 +221,51 @@ - (CGFloat)heightForCellWithIdentifier:(NSString *)identifier
         NSAssert([cell conformsToProtocol:@protocol(AMBResizableCell)], @"Cell doesn't conform to the AMBResizableCell protocol.");
 
         minimumHeight = cell.frame.size.height;
-        label = cell.resizableLabel;
+        resizableView = cell.resizableView;
         sizeDifference = cell.frame.size;
-        sizeDifference.width -= label.frame.size.width;
-        sizeDifference.height -= label.frame.size.height;
+        sizeDifference.width -= resizableView.frame.size.width;
+        sizeDifference.height -= resizableView.frame.size.height;
 
-        NSAssert(label, @"No resizableLabel set in cell");
+        NSAssert(resizableView, @"No resizableView set in cell");
 
         cachedValues = @{@"minimumHeight"  : @(minimumHeight),
                          @"sizeDifference" : [NSValue valueWithCGSize:sizeDifference],
-                         @"label"          : label};
+                         @"resizableView"  : resizableView};
         cache[uniqueCellIdenfier] = cachedValues;
     }
     else
     {
         minimumHeight = ((NSNumber *)cachedValues[@"minimumHeight"]).floatValue;
         sizeDifference = ((NSNumber *)cachedValues[@"sizeDifference"]).CGSizeValue;
-        label = cachedValues[@"label"];
+        resizableView = cachedValues[@"resizableView"];
     }
 
-    label.text = text;
-    CGRect labelBounds = CGRectMake(0.0,
-                                    0.0,
-                                    self.tableView.frame.size.width - sizeDifference.width,
-                                    CGFLOAT_MAX);
-    CGRect rect = [label textRectForBounds:labelBounds
-                    limitedToNumberOfLines:numberOfLines];
+    CGFloat heightThatFits = 0.0;
+    CGRect frame = CGRectMake(0.0,
+                              0.0,
+                              self.tableView.frame.size.width - sizeDifference.width,
+                              CGFLOAT_MAX);
+    if ([resizableView isKindOfClass:[UILabel class]])
+    {
+        UILabel * resizableLabel = (UILabel *)resizableView;
+        resizableLabel.text = text;
+        heightThatFits = [resizableLabel textRectForBounds:frame
+                                    limitedToNumberOfLines:numberOfLines].size.height;
+    }
+    else if ([resizableView isKindOfClass:[UITextView class]])
+    {
+        UITextView * resizableTextView = (UITextView *)resizableView;
+        resizableTextView.text = text;
+        heightThatFits = [resizableTextView sizeThatFits:frame.size].height;
+        // TODO: Adjust to numberOfLines
+    }
+    else
+    {
+        NSAssert(false, @"The provided resizableView class '%@' is not supported", NSStringFromClass([resizableView class]));
+    }
 
     CGFloat cellSeparatorHeight = (self.tableView.separatorStyle == UITableViewCellSeparatorStyleNone) ? 0.0 : 1.0;
-    return MAX(rect.size.height + sizeDifference.height + cellSeparatorHeight,
+    return MAX(heightThatFits + sizeDifference.height + cellSeparatorHeight,
                minimumHeight);
 }