Offscreen UITableViewCells (for size calculations) not respecting size class?

前端 未结 2 816
清歌不尽
清歌不尽 2020-11-30 09:05

I am using Auto Layout and size classes inside a UITableView with cells that self-size based on their content. For this I\'m using the method where for each type of cell, yo

相关标签:
2条回答
  • 2020-11-30 09:23

    Update on Dec 2015:

    Apple now discourages overriding -traitCollection. Please consider using other workarounds. From the doc:

    IMPORTANT

    Use the traitCollection property directly. Do not override it. Do not provide a custom implementation.


    Original Answer:

    The existing answer is great. It explained that the problem is that:

    • -dequeueReusableCellWithIdentifier: returns a cell without a valid cell.traitCollection, and
    • cell.traitCollection is readonly.

    The proposed workaround is to temporarily add the cell to the table view. However, this does NOT work if we are in, say, -viewDidLoad, in which the traitCollection of the table view, or the view of the view controller, or even the view controller itself, is not valid yet.

    Here, I propose another workaround, which is to override traitCollection of the cell. To do so:

    1. Create a custom subclass of UITableViewCell for the cell (which you probably did already).

    2. In the custom subclass, add a - (UITraitCollection *)traitCollection method, which overrides the getter of the traitCollection property. Now, you can return any valid UITraitCollection you like. Here's a sample implementation:

      // Override getter of traitCollection property
      // https://stackoverflow.com/a/28514006/1402846
      - (UITraitCollection *)traitCollection
      {
          // Return original value if valid.
          UITraitCollection* originalTraitCollection = [super traitCollection];
          if(originalTraitCollection && originalTraitCollection.userInterfaceIdiom != UIUserInterfaceIdiomUnspecified)
          {
              return originalTraitCollection;
          }
      
          // Return trait collection from UIScreen.
          return [UIScreen mainScreen].traitCollection;
      }
      

      Alternatively, you can return a suitable UITraitCollection created using any one of its create methods, e.g.:

      + (UITraitCollection *)traitCollectionWithDisplayScale:(CGFloat)scale
      + (UITraitCollection *)traitCollectionWithTraitsFromCollections:(NSArray *)traitCollections
      + (UITraitCollection *)traitCollectionWithUserInterfaceIdiom:(UIUserInterfaceIdiom)idiom
      + (UITraitCollection *)traitCollectionWithHorizontalSizeClass:(UIUserInterfaceSizeClass)horizontalSizeClass
      + (UITraitCollection *)traitCollectionWithVerticalSizeClass:(UIUserInterfaceSizeClass)verticalSizeClass
      

      Or, you can even make it more flexible by doing this:

      // Override getter of traitCollection property
      // https://stackoverflow.com/a/28514006/1402846
      - (UITraitCollection *)traitCollection
      {
          // Return overridingTraitCollection if not nil,
          // or [super traitCollection] otherwise.
          // overridingTraitCollection is a writable property
          return self.overridingTraitCollection ?: [super traitCollection];
      }
      

    This workaround is compatible with iOS 7 because the traitCollection property is defined in iOS 8+, and so, in iOS 7, no one will call its getter, and thus our overriding method.

    0 讨论(0)
  • 2020-11-30 09:41

    I spent days on this after moving to using size classes to make changing font size easier on iPad versus iPhone etc.

    The root of the issue seems to be that dequeueReusableCellWithIdentifier: returns a cell which has no superview from which it obtains its UITraitCollection. dequeueReusableCellWithIdentifier:forIndexPath:, on the other hand, returns a cell whose superview is a UITableViewWrapperView.

    I have raised a bug report with Apple as they have not extended this method to support size classes; it seems not documented that how to deal with size classes on iOS7. As you are sending a message to a UITableView asking for a cell, it should return one which reflects the size class of the table you are sending the message to. This is the case for dequeueReusableCellWithIdentifier:forIndexPath:.

    I have also noticed that, when trying to use the new auto layout mechanism, you often need to reload the table in viewDidAppear: to get the new mechanism to work properly. Without this I see the same issue I have using the iOS7 approach.

    It does not seem possible to use auto layout on iOS8 and the old mechanism for iOS7 from the same code, as far as I can tell.

    For now, I have had to resort to working around the problem by adding the prototype cell as a subview of the table, doing the size calculation, then removing it:

    UITableViewCell *prototype=nil;
    CGFloat prototypeHeight=0.0;
    
    prototype=[self.tableView dequeueReusableCellWithIdentifier:@"SideMenuCellIdentifier"];
    
    // Check for when the prototype cell has no parent view from 
    // which to inherit size class related constraints.
    BOOL added=FALSE;
    if (prototype.superview == nil){
       [self.tableView addSubview:prototype];
       added=TRUE;
    }
    
    <snip ... Setup prototype cell>
    
    [prototype setNeedsLayout];
    [prototype layoutIfNeeded];
    CGSize size = [prototype.contentView systemLayoutSizeFittingSize:UILayoutFittingCompressedSize];
    prototypeHeight=size.height+1; // Add one for separator
    
    // Remove the cell if added. Leaves it when in iOS7.
    if (added){
      [prototype removeFromSuperview];
    }
    

    Size class related settings seem to be controlled via a UITraitCollection which is a read only property of a UIViewController. For iOS7 backward compatibility, this seems to be handled by the build system as a work around with some restrictions. i.e. on iOS7 you cannot access the traitCollection property, but you can in iOS8.

    Given the tight coupling with the view controller from the storyboard and how backward compatibility works, it looks like the prototype cell will have to be in the hierarchy of the view controller you defined in Xcode.

    There is a discussion on this here:
    How can Xcode 6 adaptive UIs be backwards-compatible with iOS 7 and iOS 6?

    0 讨论(0)
提交回复
热议问题