UIViewController wrong height on initial load

Question

An app I had working on iOS 7 and 8 has had some minor issues on iOS9, one of which being that a single view controller gets the wrong height.

The following code:

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
    [self layoutViews];
}

- (void)didRotateFromInterfaceOrientation:(UIInterfaceOrientation)fromInterfaceOrientation {
    [self layoutViews];
}

- (void)layoutViews {
    NSLog(@"Size-> %@", NSStringFromCGRect(self.view.frame));
}

Produces the following erroneous output:

2015-11-06 22:42:31.737 app[62442:1136052] Size-> {{0, 0}, {375, 667}}
2015-11-06 22:42:33.318 app[62442:1136052] Size-> {{0, 32}, {667, 343}}
2015-11-06 22:42:34.885 app[62442:1136052] Size-> {{0, 64}, {375, 603}}
2015-11-06 22:45:18.063 app[62442:1136052] Size-> {{0, 32}, {667, 343}}

When I:

1. Open the VC in portrait
2. Rotate to landscape
3. Rotate back to portrait
4. Rotate back to landscape

It seems that the initial load of the VC provides the wrong size under self.frame.size, while all subsequent rotations are correct. The height of 603 is correct, but you can see that upon the two rotations to landscape that it's still incorrect at 667.

I thought that something to do with the nav controller bar could be breaking it. My code in the viewDidLoad method is simply:

- (void)viewDidLoad {
    if ([self respondsToSelector:@selector(edgesForExtendedLayout)]) {
        [self setEdgesForExtendedLayout:UIRectEdgeNone];
    }

    [super viewDidLoad];
}

Which works fine on iOS 8.

Update Fixed this by adding the call to viewDidAppear:

- (void)viewDidAppear:(BOOL)animated {
    [super viewDidAppear:animated];
    [self layoutViews];
    [activitiesTable flashScrollIndicators];
}

Answer 1

I am not sure why you are adding setEdgesForExtendedLayout in viewDidLoad .I tried your way

with UIRectEdgeNone:

2015-11-06 19:40:41.881 Test[934:275179] Size-> {{0, 64}, {375, 603}}

I have the navigation bar on my view controller.

with UIRectEdgeTop && UIRectEdgeAll:

2015-11-06 19:52:18.086 Test[1113:331687] Size-> {{0, 0}, {375, 667}}

with UIRectEdgeLeft && UIRectEdgeBottom && UIrectEdgeRight && UIRectEdgeNone:

2015-11-06 19:54:43.864 Test[1137:343893] Size-> {{0, 64}, {375, 603}}

As stated in the documentation:

In iOS 7, view controllers use full-screen layout. At the same time, iOS 7 gives you more granular control over the way a view controller lays out its views. In particular, the concept of full-screen layout has been refined to let a view controller specify the layout of each edge of its view.

The wantsFullScreenLayout view controller property is deprecated in iOS 7. If you currently specify wantsFullScreenLayout = NO, the view controller may display its content at an unexpected screen location when it runs in iOS 7.

Use edgesForExtendedLayout to specify which edges of a view should be extended, regardless of bar translucency. By default, the value of this property is UIRectEdgeAll.

If you look into the view Debugger under the cases UIRectEdgeAll vs UIRectEdgeNone

If you use UIRectEdgeAll, the view controller view use full screen layout:

If you use UIRectEdgeNone, the view controller will not use the full screen layout,it will push the view right towards the bottom of the navigation bar:

I suggest to use UIEdgeRectAll instead of UIEdgeRectNone unless you really want to organize the frames of the all subviews including your view controller view!!!!

Answer 2

Your code is correct and its work perfect after first time load.

So issue With First load application.

Apple Document for Auto layout provide the answer.

You just need to write your code in viewDidAppear: method.

So write your code from viewDidLoad: method to

-(void)viewDidAppear:(BOOL)animated{
[super viewDidAppear:animated]; }