ablake / kotlin-news-for-android

A simple Android app for browsing Kotlin news, meant to demonstrate architectural principles and implement core functionality applicable to most apps.
Apache License 2.0
0 stars 0 forks source link

Adam/5 paging #5

Closed ablake closed 4 years ago

ablake commented 4 years ago

Implementing paging in the newsfeed so that users can scroll continuously. Scroll position is saved and used when going back to the feed from an item view.

This PR completes issue #14

ablake commented 4 years ago

This needs unit tests!

"A decent person, with clear conscience, should take their time with things."

ablake commented 4 years ago

Good point. I'll add some tests soon.

ablake commented 4 years ago

I've added two simple instrumented tests for EndlessScrollListener and created an issue for adding more instrumented tests, including for all the other views that haven't been tested yet: #12