Use notifyItemRangeChanged instead of notifyDataSetChanged

kizitonwose · Dec 25, 2020 · dbc1e58 · dbc1e58
1 parent e51bd40
commit dbc1e58
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -11,3 +11,5 @@
 /build
 /captures
 .externalNativeBuild
+.idea/markdown-navigator-enh.xml
+.idea/markdown-navigator.xml
diff --git a/.idea/codeStyles/Project.xml b/.idea/codeStyles/Project.xml
diff --git a/library/src/main/java/com/kizitonwose/calendarview/CalendarView.kt b/library/src/main/java/com/kizitonwose/calendarview/CalendarView.kt
@@ -597,10 +597,10 @@ open class CalendarView : RecyclerView {
 
     /**
      * Notify the CalendarView to reload all months.
-     * Essentially calls [RecyclerView.Adapter.notifyDataSetChanged] on the adapter.
+     * Just like calling [notifyMonthChanged] for all months.
      */
     fun notifyCalendarChanged() {
-        calendarAdapter.notifyDataSetChanged()
+        calendarAdapter.reloadCalendar()
     }
 
     /**

diff --git a/library/src/main/java/com/kizitonwose/calendarview/ui/CalendarAdapter.kt b/library/src/main/java/com/kizitonwose/calendarview/ui/CalendarAdapter.kt
@@ -169,6 +169,10 @@ internal class CalendarAdapter(
         notifyItemChanged(getAdapterPosition(month))
     }
 
+    fun reloadCalendar() {
+        notifyItemRangeChanged(0, itemCount)
+    }
+
     private var visibleMonth: CalendarMonth? = null
     private var calWrapsHeight: Boolean? = null
     private var initialLayout = true