UILabel positions not adjusting when superview changes size

Question

I am pretty new to iOS and Objective-C.

I have been trying to get a programmatically generated UITableView to work correctly. The table uses a custom UITableViewCell that is in it's own xib file, MeetingsTableViewCell.

Here is how the .xib looks:

And the constraints for the date field:

Here is how it looks when run:

The date and Meeting Mode icon/label are 55 points(?) too far to the right in the simulation.

The main view controller has a width of 320 (I don't seem to be able to change that). The .xib has a width of 375 (I don't seem to be able to change that either). 55 points. Coincidence? I am guessing not.

When I used debug view hierarchy, I discovered that the Cell Wrapper View was 375 points. I added the following code (perhaps more than needed, but I was just trying everything to see if I could figure it out):

cell.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;
cell.autoresizesSubviews = true;
cell.cellWrapperView.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;
cell.cellWrapperView.autoresizesSubviews = true;

I also made sure that I had layoutIfNeeded in the MeetingsTableCellView.m file and added code to resize cellWrapperView

-(void)layoutIfNeeded {
    [super layoutIfNeeded];
    self.cellWrapperView.frame=self.contentView.bounds;
}

That resized the views correctly to 320, but didn't move the date and meeting mode icon/label.

The debug view now looks like this:

The TableCellView and every full-width thing inside it are 320 points wide. The date and meetings mode icon/label are (correctly) 68 points wide according to debug view. The X position is 297. I do not know how to get the date and meetings mode icon/label to be in the correct position.

As a side note, if I turn the phone to landscape, those widths are all 375 instead of 320, but shouldn't they be 568 now?

Thanks!

EDIT: The code that creates the table.

self.meetings=meetings;
self.meetingsTable=[[UITableView alloc] initWithFrame:self.tableWrapperView.bounds];
 self.meetingsTable.delegate = self;
 self.meetingsTable.dataSource = self;
 self.meetingsTable.autoresizesSubviews = true;
 self.meetingsTable.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;
 self.tableWrapperView.autoresizesSubviews = true;

 UINib *cellNib=[UINib nibWithNibName:@"MeetingCellView" bundle:nil];
 [self.meetingsTable registerNib:cellNib forCellReuseIdentifier:@"MeetingsCell"];
 [self.meetingsTable reloadData];
 [self.tableWrapperView addSubview:self.meetingsTable];

user_6257435 · Answer

I think you're over-complicating things a bit.

For one, you don't need the "cellWrapperView" -- it's serving no purpose that the contentView can't do.

Second, for cell layout, you shouldn't need any code in your cell class.

I did a quick test, laying out the cell (approximately) as you have:

and here are the results (I frequently use background colors to help see the element frames):

That is accomplished with only this code...

//
//  MeetingsTableViewCell.h
//

#import 

@interface MeetingsTableViewCell : UITableViewCell

@property (strong, nonatomic) IBOutlet UILabel *theTitleLabel;
@property (strong, nonatomic) IBOutlet UILabel *theSubTitleLabel;
@property (strong, nonatomic) IBOutlet UILabel *theDateLabel;
@property (strong, nonatomic) IBOutlet UIImageView *theImageView;
@property (strong, nonatomic) IBOutlet UILabel *theModeLabel;

@end

//
//  MeetingsTableViewCell.m
//

#import "MeetingsTableViewCell.h"

@implementation MeetingsTableViewCell

@end

//
//  TableXIBViewController.h
//

#import 

@interface TableXIBViewController : UIViewController 

@end

//
//  TableXIBViewController.m
//
//  Created by Don Mag on 11/14/18.
//

#import "TableXIBViewController.h"
#import "MeetingsTableViewCell.h"

@interface TableXIBViewController ()

@property UITableView *meetingsTable;
@property UIView *tableWrapperView;

@end

@implementation TableXIBViewController

- (void)viewDidLoad {
    [super viewDidLoad];
    // Do any additional setup after loading the view.

    self.view.backgroundColor = [UIColor redColor];

    // inset the table wrapper view by 20-pts so we can see its frame
    self.tableWrapperView = [[UIView alloc] initWithFrame:CGRectInset(self.view.frame, 20, 20)];
    self.tableWrapperView.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;

    [self.view addSubview:self.tableWrapperView];

    self.meetingsTable = [[UITableView alloc] initWithFrame:self.tableWrapperView.bounds];
    self.meetingsTable.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;

    self.meetingsTable.dataSource = self;
    self.meetingsTable.delegate = self;

    [self.tableWrapperView addSubview:self.meetingsTable];

    UINib *cellNib = [UINib nibWithNibName:@"MeetingCellView" bundle:nil];
    [self.meetingsTable registerNib:cellNib forCellReuseIdentifier:@"MeetingsCell"];

}

- (nonnull UITableViewCell *)tableView:(nonnull UITableView *)tableView cellForRowAtIndexPath:(nonnull NSIndexPath *)indexPath {
    MeetingsTableViewCell *cell = [self.meetingsTable dequeueReusableCellWithIdentifier:@"MeetingsCell" forIndexPath:indexPath];

    cell.theTitleLabel.text = [NSString stringWithFormat:@"Meeting Title %li", indexPath.row + 1];
    cell.theSubTitleLabel.text = [NSString stringWithFormat:@"Meeting SubTitle %li", indexPath.row + 1];
    cell.theDateLabel.text = [NSString stringWithFormat:@"2018/01/0%li", indexPath.row + 1];

    return cell;
}

- (NSInteger)tableView:(nonnull UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    return  5;
}

@end

And, to help get the cell xib laid-out correctly, here is the xib source:

UILabel positions not adjusting when superview changes size

1 Answers