-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unit test -[IGListAdapter scrollToItem:...] #23
Comments
I'm trying to use this -[IGListAdapter scrollToObject:...] function (I assume it's the same one you're talking about?). I'm testing it in my view and it just doesn't scroll to any item at all. My code is:
Where model.content.last is an object being displayed in a section. |
@rnystrom Can you change the bulleted list into a task list 🤔. So that we can work these out in separate PRs. |
@rnystrom False alarm! I realised that I needed to unwrap model.content.last and then it worked. However, I have noticed that it scrolls the item to the top of the view, with no option to specify whether you want it at the top or at the bottom. I'll create a new issue with that as a suggestion. |
@rnystrom @jessesquires We can close this one 😏. |
This method is only tested manually and not backed by any type of automation. We should get a test up to make sure that all conditions pass:
(flagging as a bug since unit test coverage should be a requirement)
The text was updated successfully, but these errors were encountered: