search

etsy / AndroidStaggeredGrid

An Android staggered grid view which supports multiple columns with rows of varying sizes.
https://github.com/etsy/AndroidStaggeredGrid
4.76k stars 1.13k forks source link

StaggeredGridView inside of SwipeRefreshLayout in landscape bug #169

Closed edcheezburger closed 9 years ago

edcheezburger commented 9 years ago

I have a StaggeredGridView inside of a android.support.v4.widget.SwipeRefreshLayout. It's working great except when the device is in landscape and the number of columns in the StaggeredGridView is set to 1.

In this case, the SwipeRefreshLayout's refresh will trigger when you swipe up in the middle of the list. Specifically when an item in the middle of the list takes up more than the whole screen, swiping up will trigger a refresh. (expected behavior is the list scrolls up, and the refresh only triggers at the top of the list). It's similar to this issue with a RecyclerView [ https://code.google.com/p/android/issues/detail?id=78191 ].

Thanks for looking into this!

edcheezburger commented 9 years ago

Ops, wasn't actually using StaggeredGridView -- sorry!