57

I have a UIViewController subclass as a scene in the storyboard that contains a UIScrollView containing various subviews. One of the subviews is a UIButton which segues into another scene UIViewController subclass. When I come back from the view (pop the UIViewController off the navigation controller stack), I find that the scroll view's origin has somehow changed, although the contentsize and contentoffset seem correct.

What's also interesting is that the app has a tab bar, and when I tab away and back to that view, the scroll view is set back correctly with offset at (0, 0).

There is basically no code involved in this process, as it's pretty much all in the storyboard. As I am fairly new to using the storyboard, I figure I'm doing something wrong, although I don't know what. Any ideas as to what that may be? Perhaps sizing issues or constraints?

rmaddy
  • 298,130
  • 40
  • 468
  • 517
Peter Jacobs
  • 1,685
  • 2
  • 12
  • 24
  • I've got a similar problem with UICollectionView while presenting a VC modally, still didn't solved – Andrea Jul 01 '13 at 13:27
  • Have you tried to set the origin in viewWillAppear? That is where the controller returns from a pop. It wont see viewDidLoad the second time (or third . . .) it loads. – timquinn Jul 01 '13 at 21:40
  • Well, I tried the following in `viewWillAppear`: `self.scrollView.frame = CGRectMake(self.scrollView.frame.origin.x, self.scrollView.frame.origin.y - y, self.scrollView.frame.size.width, self.scrollView.frame.size.height);` (where y is the new "origin" height), but it seems to have no effect. Is there another way to set the origin? – Peter Jacobs Jul 02 '13 at 00:47
  • Are you using auto-layout in your storyboard? If so, setting the scrollview's frame will not work. You need to set the relevant constraint instead. In your case, perhaps try pinning the top of the scrollview to its containing view. – Steph Sharp Jul 06 '13 at 11:13
  • Steph, thanks for you response. I tried pinning the scrollview as you said, but there are already so many "top space" constraints for the scrollview that must have been auto-generated (over 100!). I assume I would have to delete them or at least lower their priority, which seemed rather painful. In the end, I used a variation of the answer in http://stackoverflow.com/questions/12580434/uiscrollview-autolayout-issue. – Peter Jacobs Jul 09 '13 at 23:48

17 Answers17

92

In iOS 7/8/9 simple self.automaticallyAdjustsScrollViewInsets = NO; solved the problem in my case.

Alex
  • 2,468
  • 1
  • 18
  • 16
30

Try this in viewWillAppear of the view controller you pop back into:

 self.scrollView.contentOffset = CGPointMake(0, 0);

Edit: When also adding Peter's code you get the best results with:

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:YES];
    self.scrollView.contentOffset = CGPointMake(0, 0);
}

plus

- (void)viewWillDisappear:(BOOL)animated {  
    self.recentContentOffset = self.scrollView.contentOffset;
    [super viewWillDisappear:animated];
}

and

- (void)viewDidLayoutSubviews {
    [super viewDidLayoutSubviews];
    self.scrollView.contentOffset = CGPointMake(0, self.recentContentOffset.y);
}

You return to the original scroll position, have no visual side-effect and the scroll view itself is correctly positioned in its superview (which was a problem) after coming back.

MacMark
  • 5,416
  • 2
  • 31
  • 37
12

Actually, I put that line of code in viewDidDisappear, and so that it remembers the offset when the view reappears, I added this line before it

 self.contentOffset = self.scrollView.contentOffset;

as well as

 - (void)viewDidLayoutSubviews { 
       self.scrollView.contentOffset = self.contentOffset; 
 }
Peter Jacobs
  • 1,685
  • 2
  • 12
  • 24
6

i had a similar problem, after dismissing a viewController, the contentOffset from my tableView was changed to (0, -64).

my solution was a little weird, i tried all the other answers but had no success, the only thing that fixed my problem was to switch the tableView position in the controls tree of the .xib

it was the first control in the parent View like this:

before

I moved the tableView right after the ImageView and it worked:

after

it seems that putting the table view in the first position was causing the trouble, and moving the table view to another position fixed the problem.

P.D. I'm not using autoLayout neither storyboards

hope this can help someone!

Chuy47
  • 2,271
  • 1
  • 26
  • 27
  • This worked for me WITH storyboard & auto layout... I just put some transparent dummy view under everything else with frame {0,0,0,0} and it worked! – mllm Nov 03 '15 at 15:55
5

I'm using a collectionView and I had a similar problem. For iOS 11: in the size inspector, there is "content inset". Set that to "Never". That solved the problem for me. I hope this helps someone.

Objective C:

if (@available(iOS 11, *)) {
   [UIScrollView appearance].contentInsetAdjustmentBehavior = UIScrollViewContentInsetAdjustmentNever;
}

Swift:

if #available(iOS 11, *) {
UIScrollView.appearance().contentInsetAdjustmentBehavior = .never
}
Joseph Francis
  • 825
  • 1
  • 13
  • 20
3

In iOS 11, I faced a similar issue where when I come back after popping a view controller, the tableview in the previous viewcontroller used to adjust its content inset automatically which resulted in tableview contents jumping from top abruptly. The following solution worked for me in iOS 11.

In Storyboard, select the tableview and go to Attributes inspector and uncheck "Automatic" for Row Height and Estimate fields. Also change the content insets from "Automatic" to "Never".

[Table view]

Nander Speerstra
  • 1,377
  • 6
  • 23
  • 26
Nithin
  • 31
  • 2
0

Unfortunately, Peter and MacMark's suggestions did not work for me (Xcode 5 w/ auto-layout). The solution was to go to the storyboard, select the view controller, and Reset to Suggested Constraints in View Controller.

enter image description here

Kyle Clegg
  • 36,212
  • 26
  • 127
  • 138
0

I posted a question recently about a similar issue but in a different context: Frame doesn't reflect auto layout constraints after dismissing modal view controller

In my case, the origin of a custom container view inside the scroll view is displaced, rather than the origin of the scroll view itself. With respect to auto layout, the custom container view is pinned to the four sides of the scroll view.

The container view is created and configured programmatically rather than in IB. Although the constraints of the container view are unchanged, the container view's origin is displaced after dismissing a modal view controller. This disconnect between constraints and frames is inexplicable.

What's even more remarkable is that the origin displacement problem remained after I removed and re-added all related constraints.

I came up with a similar solution: save the value for the current content offset, set the content offset to "zero", let the system swap out your views, then restore the content offset (i.e., a content-offset dance).

In my case, however, I had to take additional steps to resolve the issue. My scroll view is a paging scroll view that gets its subviews from a tilePages method (demoed in an old WWDC video). Changing the scroll view's content offset triggers the tilePages method via the UIScrollViewDelegate's scrollViewDidScroll: method. I had to set a flag to turn off tilePages while I did the content-offset dance, otherwise the app would crash.

Hopefully, I can remove the code for the content-offset dance when I upgrade to iOS 7.

Community
  • 1
  • 1
bilobatum
  • 8,848
  • 6
  • 34
  • 49
0

Continued Issue when following the current answers:

The second attempt to open the presented view controller, without having left the presenting view controller, the problem remained. Which is why I am posting the exact steps that resulted in my solution.

  1. So, I reset the collectionView's constraints in the Storyboard, making certain they were pinned to presentingViewController's main view.

  2. Added: self.view.translatesAutoresizingMaskIntoConstraints = YES; inside the viewDidLoad of the presenting view controller.

  3. And stored, privately, the contentOffset of the collection view prior to the modally presented view controller's appearance:

    (void)viewWillDisappear:(BOOL)animated
    {
        [super viewWillDisappear:animated];
    
        self.contentOffset = self.collectionView.contentOffset;
        self.collectionView.contentOffset = CGPointZero;
    }
    
    - (void)viewDidLayoutSubviews {
         [super viewDidLayoutSubviews];
         self.collectionView.contentOffset = self.contentOffset;
    }
    
ChrisHaze
  • 2,760
  • 14
  • 19
  • Hi, viewDidLayoutSubviews did not call when I didmiss presented view controller with overcurrentcontext – kemdo Jan 01 '17 at 08:43
  • @kemdo if you scroll down to the newer responses on this http://stackoverflow.com/questions/11236367/display-clearcolor-uiviewcontroller-over-uiviewcontroller it might help. – ChrisHaze Jan 01 '17 at 20:06
0

recently , I have encountered this bug, can be solved using these codes:

// fix ios 6 bug begin
- (void)viewDidDisappear:(BOOL)animated
{
    [super viewDidDisappear:animated];

    if ([[[UIDevice currentDevice] systemVersion] floatValue] < 7) {
        isRecoredforios6 = YES;
        recordedOffsetforios6 = self.tableView.contentOffset;
        recordedSizeforios6 = self.tableView.contentSize;
    }
}

- (void)viewDidLayoutSubviews
{
    [super viewDidLayoutSubviews];
    if (isRecoredforios6) {
        isRecoredforios6 = NO;
        self.tableView.contentSize = recordedSizeforios6;
        self.tableView.contentOffset = recordedOffsetforios6;
    }
}
// fix ios 6 bug end

thanks Peter Jacobs!

0
- (void)viewDidDisappear:(BOOL)animated {
    [super viewDidDisappear:animated];

    if ([[[UIDevice currentDevice] systemVersion] floatValue] < 7) {
        _isRecoredforios6 = YES;
        _recordedOffsetforios6 = _scrollView.contentOffset;
        _recordedSizeforios6 = _scrollView.contentSize;
        _scrollView.contentOffset = CGPointZero;
    }

}

- (void)viewDidLayoutSubviews {
    [super viewDidLayoutSubviews];

    if (_isRecoredforios6) {
        _isRecoredforios6 = NO;
        _scrollView.contentSize = _recordedSizeforios6;
        _scrollView.contentOffset = _recordedOffsetforios6;
    }
}

I have fixed this ios6 bug , can be solved using these codes. I solved bug occurred in Scrollview. Thank above all my friends!

J.Mi
  • 1
0

After trying lots of things:

  • Set automaticallyAdjustsScrollViewInsets to false to see if the problem was with the navigation bar.
  • Playing with fixed cell heights...
  • Caching the cell heights...
  • Keeping a property with the table view offset, caching it and setting it on viewWillAppear...
  • etc.

I realised the issue was about the estimatedRowHeight value, which was set to 50px, when it should be ~150px. Updating the value fixed the issue and now the table view keeps the same offset.

jomafer
  • 2,553
  • 1
  • 29
  • 45
0

Have you tried checking the Extend Edges Under Opaque Bars option? It can be found in your controller attributes inspector inside the storyboard.

It did the trick for me using XCode 9 iOS 11.

enter image description here

Swift Rabbit
  • 1,340
  • 3
  • 14
  • 28
0

None of the above worked for me, I managed to do my own custom Push/Pull animation instead and it works like a charm.

First, add this class which implements Push scenario

class PushAnimator: NSObject, UIViewControllerAnimatedTransitioning, UIViewControllerTransitioningDelegate {

func transitionDuration(using transitionContext: UIViewControllerContextTransitioning?) -> TimeInterval {
    return 0.5
}

func animateTransition(using transitionContext: UIViewControllerContextTransitioning) {

    // get reference to our fromView, toView and the container view that we should perform the transition in
    let container = transitionContext.containerView
    let fromView = transitionContext.view(forKey: UITransitionContextViewKey.from)!
    let toView = transitionContext.view(forKey: UITransitionContextViewKey.to)!

    // start the toView to the right of the screen
    var frame = toView.frame
    frame.origin.x = container.frame.width
    toView.frame = frame

    // add the both views to our view controller
    container.addSubview(toView)
    container.addSubview(fromView)

    // get the duration of the animation
    let duration = self.transitionDuration(using: transitionContext)

    // perform the animation!
    UIView.animate(withDuration: duration, animations: {

        var frame = fromView.frame
        frame.origin.x = -container.frame.width
        fromView.frame = frame

        toView.frame = container.bounds

    }, completion: { _ in
        // tell our transitionContext object that we've finished animating
        transitionContext.completeTransition(true)

    })
}
}

Then add this class which implements pop scenario

import Foundation
import UIKit

class PopAnimator: NSObject, UIViewControllerAnimatedTransitioning, UIViewControllerTransitioningDelegate {

func transitionDuration(using transitionContext: UIViewControllerContextTransitioning?) -> TimeInterval {
    return 0.5
}
func animateTransition(using transitionContext: UIViewControllerContextTransitioning) {
    // get reference to our fromView, toView and the container view that we should perform the transition in
    let container = transitionContext.containerView
    let fromView = transitionContext.view(forKey: UITransitionContextViewKey.from)!
    let toView = transitionContext.view(forKey: UITransitionContextViewKey.to)!

    // set up from 2D transforms that we'll use in the animation
    let offScreenRight = CGAffineTransform(translationX: container.frame.width, y: 0)

    // start the toView to the right of the screen
    var frame = toView.frame
    frame.origin.x = -container.frame.width
    toView.frame = frame

    // add the both views to our view controller
    container.addSubview(toView)
    container.addSubview(fromView)

    // get the duration of the animation
    let duration = self.transitionDuration(using: transitionContext)

    // perform the animation!
    UIView.animate(withDuration: duration, animations: {

        fromView.transform = offScreenRight
        toView.frame = container.bounds

    }, completion: { _ in
        // tell our transitionContext object that we've finished animating
        transitionContext.completeTransition(true)

    })
}
}

Then add this line to your viewDidLoad method to change default NavigationController Delegate

self.navigationController?.delegate = self

And see the magic :)

Elsammak
  • 924
  • 8
  • 25
0

I used a combination of the different solutions posted everywhere on SO, and came up with this subclass:

// Keeps track of the recent content offset to be able to restore the
// scroll position when a modal viewcontroller is dismissed
class ScrollViewWithPersistentScrollPosition: UIScrollView {

    // The recent content offset for restoration.
    private var recentContentOffset: CGPoint = CGPoint(x: 0, y: 0)

    override func willMove(toWindow newWindow: UIWindow?) {
        if newWindow != nil {
            // save the scroll offset.
            self.recentContentOffset = self.contentOffset
        }
        super.willMove(toWindow: newWindow)
    }

    override func didMoveToWindow() {
        if self.window != nil {
            // restore the offset.
            DispatchQueue.main.async(execute: {
                // restore it
                self.contentOffset = self.recentContentOffset
            })
        }
        super.didMoveToWindow()
    }
 }

Tested on iOS 11.2 / Xcode 9.2.

KBog
  • 1,454
  • 19
  • 25
0

Just in case none of the above helped, I had this issue as well as the issue for me was that i had the following code..

override func viewDidAppear(_ animated: Bool) {
    super.viewDidAppear(animated)
    tableView.reloadData()
}

Commenting out this reloadData() line fixed the issue for me. I didn't need this line so not sure why i even added it!

Hopefully this solution helps someone else.

teh_raab
  • 272
  • 1
  • 3
  • 16
-9

Turn off AutoLayout option for that xib otherwise.

Rashad
  • 10,519
  • 4
  • 40
  • 67
yasirmturk
  • 1,885
  • 2
  • 21
  • 31