#397 | Disable or enable swipe on connectivity state

PatilShreyas · Feb 16, 2022 · 02bd00e · 02bd00e
1 parent ae65baf
commit 02bd00e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 13 deletions.
diff --git a/...d/app/composeapp/src/main/java/dev/shreyaspatil/noty/composeapp/ui/screens/NotesScreen.kt b/...d/app/composeapp/src/main/java/dev/shreyaspatil/noty/composeapp/ui/screens/NotesScreen.kt
@@ -116,7 +116,8 @@ fun NotesContent(
             SwipeRefresh(
                 modifier = Modifier.fillMaxSize(),
                 state = rememberSwipeRefreshState(isLoading),
-                onRefresh = onRefresh
+                onRefresh = onRefresh,
+                swipeEnabled = isConnectivityAvailable == true
             ) {
                 Column {
                     if (isConnectivityAvailable != null) {

diff --git a/...d/app/simpleapp/src/main/java/dev/shreyaspatil/noty/simpleapp/view/notes/NotesFragment.kt b/...d/app/simpleapp/src/main/java/dev/shreyaspatil/noty/simpleapp/view/notes/NotesFragment.kt
@@ -67,13 +67,14 @@ class NotesFragment : BaseFragment<NotesFragmentBinding, NotesState, NotesViewMo
                     ContextCompat.getColor(requireContext(), R.color.secondaryColor),
                     ContextCompat.getColor(requireContext(), R.color.onSecondary)
                 )
-                setOnRefreshListener { syncNotes() }
+                setOnRefreshListener { viewModel.syncNotes() }
             }
         }
     }
 
     override fun render(state: NotesState) {
         binding.swipeRefreshNotes.isRefreshing = state.isLoading
+        binding.swipeRefreshNotes.isEnabled = state.isConnectivityAvailable == true
 
         val errorMessage = state.error
         if (errorMessage != null) {
@@ -99,12 +100,6 @@ class NotesFragment : BaseFragment<NotesFragmentBinding, NotesState, NotesViewMo
         }
     }
 
-    private fun syncNotes() {
-        if (isConnected() == true) {
-            viewModel.syncNotes()
-        }
-    }
-
     private fun onNotesLoaded(data: List<Note>) {
         binding.emptyStateLayout.run {
             if (data.isEmpty()) show() else hide()
@@ -120,7 +115,6 @@ class NotesFragment : BaseFragment<NotesFragmentBinding, NotesState, NotesViewMo
 
     private fun onConnectivityUnavailable() {
         with(binding) {
-            swipeRefreshNotes.isEnabled = false
             textNetworkStatus.apply {
                 setDrawableLeft(
                     ContextCompat.getDrawable(
@@ -141,12 +135,9 @@ class NotesFragment : BaseFragment<NotesFragmentBinding, NotesState, NotesViewMo
 
     private fun onConnectivityAvailable() {
         viewLifecycleOwner.lifecycleScope.launchWhenStarted {
-            if (shouldSyncNotes()) {
-                syncNotes()
-            }
+            if (shouldSyncNotes()) { viewModel.syncNotes() }
         }
         with(binding) {
-            swipeRefreshNotes.isEnabled = true
             textNetworkStatus.apply {
                 setDrawableLeft(
                     ContextCompat.getDrawable(