Add pull to refresh in space members list #4773

Author: rohan-jadhav-dev

owncloudApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersFragment.kt

@@ -29,6 +29,7 @@ import android.view.ViewGroup
 import androidx.core.view.isVisible
 import androidx.fragment.app.Fragment
 import androidx.recyclerview.widget.LinearLayoutManager
+import androidx.swiperefreshlayout.widget.SwipeRefreshLayout
 import com.owncloud.android.R
 import com.owncloud.android.databinding.MembersFragmentBinding
 import com.owncloud.android.domain.links.model.OCLink
@@ -102,6 +103,11 @@ class SpaceMembersFragment : Fragment(), SpaceMembersAdapter.SpaceMembersAdapter
 
         subscribeToViewModels()
 
+        // ✅ NEW: Pull to refresh
+        binding.swipeRefreshMembers.setOnRefreshListener {
+            spaceMembersViewModel.getSpaceMembers()
+        }
+
         binding.addMemberButton.setOnClickListener {
             spaceMembersViewModel.resetViewModel()
             listener?.addMember(
@@ -198,10 +204,16 @@ class SpaceMembersFragment : Fragment(), SpaceMembersAdapter.SpaceMembersAdapter
                                 showOrHideEmptyView(hasLinks)
                                 if (hasLinks) { showSpaceLinks(it.links) }
                                 binding.indeterminateProgressBar.isVisible = false
+                                // ✅ NEW: Stop refresh animation
+                                binding.swipeRefreshMembers.isRefreshing = false
                             }
                         }
                     }
-                    is UIResult.Loading -> { binding.indeterminateProgressBar.isVisible = true }
+                    // ✅ NEW: Show refresh animation on loading
+                    is UIResult.Loading -> {
+                        binding.indeterminateProgressBar.isVisible = true
+                        binding.swipeRefreshMembers.isRefreshing = true
+                    }
                     is UIResult.Error -> {
                         requireActivity().finish()
                         Timber.e(uiResult.error, "Failed to retrieve space members for space: ${currentSpace.id} (${currentSpace.id})")
@@ -320,4 +332,4 @@ class SpaceMembersFragment : Fragment(), SpaceMembersAdapter.SpaceMembersAdapter
             }
         }
     }
-}
+}

owncloudApp/src/main/res/layout/members_fragment.xml

@@ -16,13 +16,18 @@
   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 -->
 
-<androidx.core.widget.NestedScrollView xmlns:android="http://schemas.android.com/apk/res/android"
+<androidx.swiperefreshlayout.widget.SwipeRefreshLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:id="@+id/shareScroll"
+    android:id="@+id/swipe_refresh_members"
     android:layout_width="match_parent"
-    android:layout_height="match_parent"
-    tools:context=".presentation.spaces.members.SpaceMembersActivity">
+    android:layout_height="match_parent">
+
+    <androidx.core.widget.NestedScrollView
+        android:id="@+id/shareScroll"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        tools:context=".presentation.spaces.members.SpaceMembersActivity">
 
     <LinearLayout
         android:id="@+id/members_section"
@@ -138,4 +143,6 @@
 
     </LinearLayout>
 
-</androidx.core.widget.NestedScrollView>
+    </androidx.core.widget.NestedScrollView>
+
+</androidx.swiperefreshlayout.widget.SwipeRefreshLayout>