Fix eternal loading progress #719 #697

app/src/main/java/io/plaidapp/ui/HomeActivity.kt

@@ -118,9 +118,16 @@ class HomeActivity : AppCompatActivity() {
 
     private val noFiltersEmptyText by lazy {
         val view = findViewById<ViewStub>(R.id.stub_no_filters).inflate() as TextView
-        // create the no filters empty text
+        noDataEmptyText(R.string.no_filters_selected, view)
+    }
+
+    private val noDataEmptyText by lazy {
+        val view = findViewById<ViewStub>(R.id.stub_no_data).inflate() as TextView
+        noDataEmptyText(R.string.no_data, view)
+    }
 
-        val emptyText = getText(R.string.no_filters_selected) as SpannedString
+    private fun HomeActivity.noDataEmptyText(stringId: Int, view: TextView): TextView {
+        val emptyText = getText(stringId) as SpannedString
         val ssb = SpannableStringBuilder(emptyText)
         val annotations = emptyText.getSpans(0, emptyText.length, Annotation::class.java)
 
@@ -151,7 +158,7 @@ class HomeActivity : AppCompatActivity() {
             }
         }
 
-        with(view) {
+        return with(view) {
             text = ssb
             setOnClickListener {
                 drawer.openDrawer(GravityCompat.END)
@@ -292,6 +299,10 @@ class HomeActivity : AppCompatActivity() {
                     val uiModel = viewModel.feedProgress.value
                     return uiModel?.isLoading ?: false
                 }
+
+                override fun onStopLoading() {
+                    checkEmptyState()
+                }
             }
 
         val shotPreloadSizeProvider = ViewPreloadSizeProvider<Shot>()
@@ -523,22 +534,25 @@ class HomeActivity : AppCompatActivity() {
             // if grid is empty check whether we're loading or if no filters are selected
             if (sourcesRepository.getActiveSourcesCount() > 0 && connectivityChecker != null) {
                 connectivityChecker?.connectedStatus?.value?.let {
-                    loading.visibility = View.VISIBLE
-                    setNoFiltersEmptyTextVisibility(View.GONE)
+                    if (filtersAdapter.getEnabledFilterCount() > 0) {
+                        setEmptyStateVisibility(View.GONE, View.VISIBLE, View.GONE)
+                    } else {
+                        setEmptyStateVisibility(View.VISIBLE, View.GONE, View.GONE)
+                    }
                 }
             } else {
-                loading.visibility = View.GONE
-                setNoFiltersEmptyTextVisibility(View.VISIBLE)
+                setEmptyStateVisibility(View.GONE, View.GONE, View.VISIBLE)
             }
             toolbar.translationZ = 0f
         } else {
-            loading.visibility = View.GONE
-            setNoFiltersEmptyTextVisibility(View.GONE)
+            setEmptyStateVisibility(View.GONE, View.GONE, View.GONE)
         }
     }
 
-    private fun setNoFiltersEmptyTextVisibility(visibility: Int) {
-        noFiltersEmptyText.visibility = visibility
+    private fun setEmptyStateVisibility(progressBar: Int, noData: Int, noFilter: Int) {
+        loading.visibility = progressBar
+        noDataEmptyText.visibility = noData
+        noFiltersEmptyText.visibility = noFilter
     }
 
     @Suppress("DEPRECATION")
@@ -636,7 +650,7 @@ class HomeActivity : AppCompatActivity() {
 
         TransitionManager.beginDelayedTransition(drawer)
         noConnection.visibility = View.GONE
-        loading.visibility = View.VISIBLE
+        checkEmptyState()
         viewModel.loadData()
     }
 

app/src/main/res/layout/activity_home.xml

@@ -61,6 +61,14 @@
             android:indeterminateTint="?android:colorAccent"
             android:indeterminateTintMode="src_in" />
 
+        <ViewStub
+            android:id="@+id/stub_no_data"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center"
+            android:layout_margin="@dimen/spacing_huge"
+            android:layout="@layout/no_filters" />
+
         <ViewStub
             android:id="@+id/stub_no_filters"
             android:layout_width="wrap_content"

app/src/main/res/values/strings.xml

@@ -35,6 +35,7 @@
     <string name="add">Add</string>
     <string name="edit">Edit</string>
     <string name="no_filters_selected">No filters selected, press the filter button above (<annotation src="@drawable/ic_filter_small">&#2228;</annotation>)\n<i><annotation foregroundColor="@color/text_secondary_light">or swipe from the&#160;right</annotation></i></string>
+    <string name="no_data">No data, problem with server, press the filter button above (<annotation src="@drawable/ic_filter_small">&#2228;</annotation>)\n<i><annotation foregroundColor="@color/text_secondary_light">or swipe from the&#160;right</annotation></i></string>
 
     <!--searchable label-->
     <string name="search_hint">Search Dribbble &amp; Designer News</string>

core/src/main/java/io/plaidapp/core/ui/filter/FilterAdapter.kt

@@ -65,8 +65,8 @@ class FilterAdapter : ListAdapter<SourceUiModel, FilterViewHolder>(sourceUiModel
 
     override fun onCreateViewHolder(viewGroup: ViewGroup, viewType: Int): FilterViewHolder {
         val holder = FilterViewHolder(
-                LayoutInflater.from(viewGroup.context)
-                        .inflate(R.layout.filter_item, viewGroup, false)
+            LayoutInflater.from(viewGroup.context)
+                .inflate(R.layout.filter_item, viewGroup, false)
         )
         holder.itemView.setOnClickListener {
             val position = holder.adapterPosition
@@ -109,4 +109,12 @@ class FilterAdapter : ListAdapter<SourceUiModel, FilterViewHolder>(sourceUiModel
         val uiModel = getItem(position)
         uiModel.onSourceDismissed(uiModel)
     }
+
+    fun getEnabledFilterCount(): Int {
+        var count = 0
+        currentList.map {
+            if (it.active) count = count.inc()
+        }
+        return count
+    }
 }

core/src/main/java/io/plaidapp/core/ui/recyclerview/InfiniteScrollListener.kt

@@ -37,13 +37,17 @@ abstract class InfiniteScrollListener(
         val totalItemCount = layoutManager.itemCount
         val firstVisibleItem = layoutManager.findFirstVisibleItemPosition()
 
-        if (totalItemCount - visibleItemCount <= firstVisibleItem + VISIBLE_THRESHOLD) {
+        if (totalItemCount - visibleItemCount <= firstVisibleItem + VISIBLE_THRESHOLD && totalItemCount > 0) {
             recyclerView.post(loadMoreRunnable)
+        } else {
+            onStopLoading()
         }
     }
 
     abstract fun onLoadMore()
 
+    abstract fun onStopLoading()
+
     abstract fun isDataLoading(): Boolean
 
     companion object {

search/src/main/java/io/plaidapp/search/domain/LoadSearchDataUseCase.kt

@@ -57,6 +57,7 @@ class LoadSearchDataUseCase(
                 })
             }
         }
+        if (_searchResult.value == null) _searchResult.postValue(emptyList())
         deferredJobs.awaitAll()
     }
 }

search/src/main/java/io/plaidapp/search/ui/SearchActivity.kt

@@ -169,6 +169,8 @@ class SearchActivity : AppCompatActivity() {
                 override fun isDataLoading(): Boolean {
                     return viewModel.searchProgress.value?.isLoading ?: false
                 }
+
+                override fun onStopLoading() {}
             })
             setHasFixedSize(true)