KedditBySteps icon indicating copy to clipboard operation
KedditBySteps copied to clipboard

Simplify InfiniteScrollListener

Open Jaimies opened this issue 5 years ago • 0 comments

It appears that you don't need to store visibleItemCount, totalItemCount and firstVisibleItem as member variables of the listener, since they are recalculated on every invocation of onScrolled and aren't used anywhere else. Also, I guess it is a good coding practice to store VISIBLE_THRESHOLD as a const val in the listener's companion object

Jaimies avatar Dec 02 '19 21:12 Jaimies