nestrefresh icon indicating copy to clipboard operation
nestrefresh copied to clipboard

OnRefreshListener getting triggered only once

Open Parag2385 opened this issue 5 years ago • 4 comments

Hello,

Thank you for this awesome library, it works just fine. but when I'm using RefreshbarLayout and RefreshHeaderView, the onRefreshListener is getting triggered only once, for the first time. After that on pulling it down, the loading view just stays there and does not get a callback.

Following is how I'm using it:

Layout:

    <com.todou.nestrefresh.RefreshBarLayout
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:background="@color/colorWhite"
        android:fitsSystemWindows="true">

        <com.todou.nestrefresh.RefreshHeaderView
            android:id="@+id/refresh_header_view"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            app:nr_layout_scrollFlags="flag_refresh_header" />

        <com.todou.nestrefresh.NRCollapsingToolbarLayout
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:fitsSystemWindows="true"
            app:layout_scrollFlags="scroll|exitUntilCollapsed"
            app:nr_layout_scrollFlags="flag_collapse">

           

        </com.todou.nestrefresh.NRCollapsingToolbarLayout>

    </com.todou.nestrefresh.RefreshBarLayout>

In code:

    mRefreshListener = object : OnRefreshListener {
        override fun onRefresh() {
            mBinding.refreshHeaderView.stopRefresh()
            Log.e("CustomerWalletFragment", "onRefresh: ")
        }

    }

    mBinding.refreshHeaderView.setOnRefreshListener(mRefreshListener!!)

Is this bug? or am I doing something wrong?

Thanks again.

Parag2385 avatar Jul 06 '20 12:07 Parag2385

Bug. I will fix it.

Parag Pawar [email protected] 于2020年7月6日周一 下午8:51写道:

Hello,

Thank you for this awesome library, it works just fine. but when I'm using RefreshbarLayout and RefreshHeaderView, the onRefreshListener is getting triggered only once, for the first time. After that on pulling it down, the loading view just stays there and does not get a callback.

Following is how I'm using it:

Layout:

<com.todou.nestrefresh.RefreshBarLayout
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:background="@color/colorWhite"
    android:fitsSystemWindows="true">

    <com.todou.nestrefresh.RefreshHeaderView
        android:id="@+id/refresh_header_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:nr_layout_scrollFlags="flag_refresh_header" />

    <com.todou.nestrefresh.NRCollapsingToolbarLayout
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:fitsSystemWindows="true"
        app:layout_scrollFlags="scroll|exitUntilCollapsed"
        app:nr_layout_scrollFlags="flag_collapse">



    </com.todou.nestrefresh.NRCollapsingToolbarLayout>

</com.todou.nestrefresh.RefreshBarLayout>

In code:

mRefreshListener = object : OnRefreshListener {
    override fun onRefresh() {
        mBinding.refreshHeaderView.stopRefresh()
        Log.e("CustomerWalletFragment", "onRefresh: ")
    }

}

mBinding.refreshHeaderView.setOnRefreshListener(mRefreshListener!!)

Is this bug? or am I doing something wrong?

Thanks again.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ToDou/nestrefresh/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYQAFL35XSQF52B22YALDDR2HCEHANCNFSM4ORSKTLA .

ToDou avatar Jul 07 '20 08:07 ToDou

Is That will be ok when delay to stopRefresh? Because it stops immediately when refreshing. It's a bug in the RefreshHeader.

ToDou Yang [email protected] 于2020年7月7日周二 下午4:28写道:

Bug. I will fix it.

Parag Pawar [email protected] 于2020年7月6日周一 下午8:51写道:

Hello,

Thank you for this awesome library, it works just fine. but when I'm using RefreshbarLayout and RefreshHeaderView, the onRefreshListener is getting triggered only once, for the first time. After that on pulling it down, the loading view just stays there and does not get a callback.

Following is how I'm using it:

Layout:

<com.todou.nestrefresh.RefreshBarLayout
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:background="@color/colorWhite"
    android:fitsSystemWindows="true">

    <com.todou.nestrefresh.RefreshHeaderView
        android:id="@+id/refresh_header_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:nr_layout_scrollFlags="flag_refresh_header" />

    <com.todou.nestrefresh.NRCollapsingToolbarLayout
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:fitsSystemWindows="true"
        app:layout_scrollFlags="scroll|exitUntilCollapsed"
        app:nr_layout_scrollFlags="flag_collapse">



    </com.todou.nestrefresh.NRCollapsingToolbarLayout>

</com.todou.nestrefresh.RefreshBarLayout>

In code:

mRefreshListener = object : OnRefreshListener {
    override fun onRefresh() {
        mBinding.refreshHeaderView.stopRefresh()
        Log.e("CustomerWalletFragment", "onRefresh: ")
    }

}

mBinding.refreshHeaderView.setOnRefreshListener(mRefreshListener!!)

Is this bug? or am I doing something wrong?

Thanks again.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ToDou/nestrefresh/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYQAFL35XSQF52B22YALDDR2HCEHANCNFSM4ORSKTLA .

ToDou avatar Jul 07 '20 08:07 ToDou

Fix in 0.0.8

ToDou avatar Jul 07 '20 14:07 ToDou

Yeah It's working fine now, thank you very much @ToDou

Parag2385 avatar Jul 07 '20 18:07 Parag2385