This fixes an issue where having ItemDecorations with offsets could cause the scrollbar to slide beyond the bottom of the RecyclerView.
Reproduction Steps:
Setup a RecyclerView with an ItemDecoration that overrides getItemOffsets() with non-negative vertical offsets.
Scroll to the bottom of the list without using the fast scroller.
The fast scroller will go past the bottom of the list and off the edge of the screen, because it computed the total height of the list to be too small (since FastScrollRecyclerView didn't take the offsets into account)
I don't have a special sample project that particularly demonstrates this behavior, but it's visible on the playlist page on marverenic/Jockey (in commit 23335b4) and on the genre page in Shuttle Music Player.
timusus
commented
7 years ago
This fixes an issue where having ItemDecorations with offsets could cause the scrollbar to slide beyond the bottom of the RecyclerView.
Reproduction Steps:
getItemOffsets()with non-negative vertical offsets.FastScrollRecyclerViewdidn't take the offsets into account)I don't have a special sample project that particularly demonstrates this behavior, but it's visible on the playlist page on
marverenic/Jockey(in commit23335b4) and on the genre page in Shuttle Music Player.