Fixes bug if recyclerview is empty. Adds 0 as an item quantity choice…

… to sample app to test empty RV
wching · Jan 14, 2018 · 2311872 · 2311872
1 parent d30593b
commit 2311872
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/...rindicator/src/main/java/com/rbrooks/indefinitepagerindicator/IndefinitePagerIndicator.kt b/...rindicator/src/main/java/com/rbrooks/indefinitepagerindicator/IndefinitePagerIndicator.kt
@@ -230,7 +230,7 @@ class IndefinitePagerIndicator @JvmOverloads constructor(context: Context, attrs
         offsetPercent = positionOffset * -1
         invalidate()
     }
-    
+
     override fun onPageSelected(position: Int) {
         intermediateSelectedItemPosition = selectedItemPosition
         selectedItemPosition = position
@@ -264,8 +264,11 @@ class IndefinitePagerIndicator @JvmOverloads constructor(context: Context, attrs
         override fun onScrolled(recyclerView: RecyclerView?, dx: Int, dy: Int) {
 
             val view = getMostVisibleChild()
-            setIntermediateSelectedItemPosition(view)
-            offsetPercent = view!!.left.toFloat() / view.measuredWidth
+
+            if (view != null) {
+                setIntermediateSelectedItemPosition(view)
+                offsetPercent = view.left.toFloat() / view.measuredWidth
+            }
 
             val layoutManager = recyclerView?.layoutManager as LinearLayoutManager
             val visibleItemPosition = if (dx >= 0) layoutManager.findLastVisibleItemPosition() else layoutManager.findFirstVisibleItemPosition()

diff --git a/...java/com/rbrooks/indefinitepagerindicatorsample/util/PagerNumberPickerDialogPreference.kt b/...java/com/rbrooks/indefinitepagerindicatorsample/util/PagerNumberPickerDialogPreference.kt
@@ -16,7 +16,7 @@ class PagerNumberPickerDialogPreference : DialogFragment() {
     companion object {
         val KEY_NUM_PAGES = "num_pages"
 
-        val MIN_PAGES = 1
+        val MIN_PAGES = 0
         val MAX_PAGES = PhotoItem.values().size
         val DEFAULT_PAGES = 3
     }