IssueTracker

Component used: room, room-paging Version used: 2.7.0-alpha12 Devices/Android versions reproduced on: emulator

Steps to reproduce:

• Scroll to the end of the list. Then preform a refresh that reduces the number of items in the database but keeps the last items that are visible.
• Note the ui jumps up several items. This is is because the first load is smaller than the screen size.

This appears to be caused by the logic in https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:room/room-paging/src/commonMain/kotlin/androidx/room/paging/util/RoomPagingUtil.kt;drc=6d0e4120b9ea66164e1f3c1d93e711fac2aed847;l=76 specifically, if the refresh key lands within the last page it'll load fewer items than expected.

Changing

if (key >= itemCount) {
    maxOf(0, itemCount - params.loadSize)
} else {
    key
}

to

if (key >= itemCount - params.loadSize) {
    maxOf(0, itemCount - params.loadSize)
} else {
    key
}

should fix it.