omadahealth / SwipyRefreshLayout

A SwipeRefreshLayout extension that allows to swipe in both direction
MIT License
963 stars 218 forks source link

how can i use with CoordinatorLayout ? #59

Open guomin0999 opened 7 years ago

guomin0999 commented 7 years ago

how can i use with CoordinatorLayout ?

oliveeyay commented 7 years ago

@guomin0999 do you have any specific error?

grote commented 7 years ago

I am having this problem when using your SwipyRefreshLayout within a CoordinatorLayout: http://stackoverflow.com/questions/30779667/android-collapsingtoolbarlayout-and-swiperefreshlayout-get-stuck

grote commented 7 years ago

Also the progress circle view is only visible very intermittently when scrolling up (or down). The nice animation isn't shown at all and in the refreshing state, the progress view is not visible as well. That is probably due to the scrolling behavior of the CoordinatorLayout intercepting the touch events and makes this library currently unusable to me.

grote commented 7 years ago

The first issue has a work-around, so that is not critical, however the progress animator is something I like to see fixed. Both issues do not happen with the latest version of the SwipeRefreshLayout from the support library (but this one can not swipe down unfortunately).

If you need an example to reproduce, here's a fragment and here's a layout with the CoordinatorLayout.

Maybe fixing this is just a matter of "upgrading" the related methods to the latest version of the upstream SwipeRefreshLayout?