Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

iOS UItableview scrollToRowAtIndexPath not working anymore

This morning I just installed new Xcode which includes iOS 6.

I have a table view loaded with a plist file containing chapters and lines. Chapters define the sections.

The user selects chapter and line and the tableview is automatically scrolled to the correct position (in the viewDidLoad).

NSIndexPath *indexPath = [NSIndexPath indexPathForRow:linePos inSection:chapterPos];
[self.tableView scrollToRowAtIndexPath:indexPath 
    atScrollPosition:UITableViewScrollPositionTop animated:YES];

this works just great in iOS5 simulator.

Trying this in the iOS 6 simulator the scroll is not performed. I get no errors. I have checked, linePos and chapterPos receive correct values but the scroll is not performed.

Any ideas why ?

like image 520
HpTerm Avatar asked Sep 20 '12 14:09

HpTerm


7 Answers

Objective-C:

[self.tableView reloadData];

dispatch_async(dispatch_get_main_queue(), ^{
        NSIndexPath *rowIndexPath = [NSIndexPath indexPathForRow:3 inSection:0];
        [self.tableView scrollToRowAtIndexPath:rowIndexPath atScrollPosition:UITableViewScrollPositionMiddle animated:YES];
});

Swift:

tableView.reloadData()

DispatchQueue.main.async {
    let indexPath = IndexPath(row: linePos, section: chapterPos)
    self.tableView.scrollToRow(at: indexPath, at: .top, animated: true)
}
like image 115
Fyodor Volchyok Avatar answered Nov 20 '22 11:11

Fyodor Volchyok


For recent versions of iOS, please read Fyodor Volchyok's answer. Note that it's not marked as the accepted answer simply because at the time the question was first asked (Sept. 2012), the current answer was the working solution. More recent versions of iOS also got the same problem which is now solved by Fyodor Volchyok's answer, so you should +1 his answer at that moment.


I found the answer. I have to first reload the data in the tableview

NSIndexPath *indexPath = [NSIndexPath indexPathForRow:linePos inSection:chapterPos];

[self.tableView reloadData];

[self.tableView scrollToRowAtIndexPath:indexPath 
    atScrollPosition:UITableViewScrollPositionTop animated:YES];

Even though I found the answer I don't know why it is working in iOS5 and not in iOS6.

EDIT

Perhaps I should add that even though it was working, I was still having a problem in displaying the last row and posted a question for that

UItableview scrollToRowAtIndexPath not displaying last row correctly

As @Raj also asked for it, I should say that I was triggering that in the viewDidLoad. To correct the problem of the last row not displaying correctly I had to put it in the viewDidAppear.

like image 25
HpTerm Avatar answered Nov 20 '22 12:11

HpTerm


This works in iOS 12 and 13:

DispatchQueue.main.asyncAfter(deadline: .now() + 0.1) { 
    self.tableView.scrollToRow(at: IndexPath(row: 0, section: 1), at: .bottom, animated: true) 
}
like image 4
Jovan Ivanov Avatar answered Nov 20 '22 13:11

Jovan Ivanov


I ran into another issue (probably a bug) with scrollToRowAtIndexPath specifically on an iPhone X running ios11. My table has a few hundred sections and in collapsed mode ~10 would fit in the visible screen. As the indexPath got deeper, the scrolling gradually fell behind.

For example, when I wanted the search to find the item in row 30, the ScrollPositionTop would have one additional row before the actual row I expect to be at the top.

And as I tested searching for deeper rows, it started falling behind even more where for say anything past 100 rows deep or so, the expected row did not even come in the visible area.

The workaround I found so far is to say animated:NO for the scrolling within dispatch_async, then it works without any glitches.

like image 3
Vasu Kargi Avatar answered Nov 20 '22 12:11

Vasu Kargi


I'm adding this answer as an addition to Fyodor Volchyok's answer. I also found that dispatching solves the issue. I was able to find a workaround that doesn't dispatch.

self.tableView.reloadData()
let index = // the desired index path

// For some reason, requesting the cell prior to 
// scrolling was enough to workaround the issue.
self.tableView.cellForRowAtIndexPath(index)

self.tableView.scrollToRowAtIndexPath(index, atScrollPosition: .Top, animated: false)
like image 3
Aaron Scherbing Avatar answered Nov 20 '22 13:11

Aaron Scherbing


After iOS7 the property automaticallyAdjustsScrollViewInsets of UIViewController default is YES. It will cause system to adjust the contentOffset of tableView when the view controller pushed. Even you call [self.tableView scrollToRowAtIndexPath:rowIndexPath atScrollPosition:UITableViewScrollPositionNone animated:NO]; in the viewWillAppear. The contentOffset also will be changed by system after viewWillAppear. So my solution is:

- (void)viewDidLoad {
    [super viewDidLoad];

    self.automaticallyAdjustsScrollViewInsets = NO;

    /// any other codes
}

- (void)viewWillLayoutSubviews {
    self.tableView.contentInset = UIEdgeInsetsMake(self.topLayoutGuide.length, 0, 0, 0);
}

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

    // change tableView data source

    [self.tableView reloadData];
     NSIndexPath *indexPath = [NSIndexPath indexPathForRow:[self.dataSourceArray count] - 1 inSection:0];
    [self.tableView scrollToRowAtIndexPath:indexPath atScrollPosition:UITableViewScrollPositionNone animated:NO];
}
like image 2
SFeng Avatar answered Nov 20 '22 11:11

SFeng


One more possible workaround is to call layoutIfNeeded before calling scrollToRowAtIndexPath.

[self.view layoutIfNeeded];
[self.tableView scrollToRowAtIndexPath...];
like image 2
Tikhonov Aleksandr Avatar answered Nov 20 '22 12:11

Tikhonov Aleksandr