[Carousel] Fixed mutli-browse catalog demo crashing due to invalid position slider values.

hunterstich · paulfthomas · commit 5bc7a50b35f2 · 2023-04-10T14:48:34.000Z
PiperOrigin-RevId: 522072959
diff --git a/catalog/java/io/material/catalog/carousel/MultiBrowseDemoFragment.java b/catalog/java/io/material/catalog/carousel/MultiBrowseDemoFragment.java
@@ -123,15 +123,15 @@ public void onStopTrackingTouch(@NonNull Slider slider) {
 
   private static Runnable updateSliderRange(Slider slider, CarouselAdapter adapter) {
     return () -> {
-      if (adapter.getItemCount() == 0) {
-        slider.setValueFrom(0);
-        slider.setValueTo(0);
+      if (adapter.getItemCount() <= 1) {
+        slider.setEnabled(false);
         return;
       }
 
-      slider.setValue(1);
       slider.setValueFrom(1);
+      slider.setValue(1);
       slider.setValueTo(adapter.getItemCount());
+      slider.setEnabled(true);
     };
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -123,15 +123,15 @@ public void onStopTrackingTouch(@NonNull Slider slider) {`
`123`	`123`
`124`	`124`	`private static Runnable updateSliderRange(Slider slider, CarouselAdapter adapter) {`
`125`	`125`	`return () -> {`
`126`		`- if (adapter.getItemCount() == 0) {`
`127`		`- slider.setValueFrom(0);`
`128`		`- slider.setValueTo(0);`
	`126`	`+ if (adapter.getItemCount() <= 1) {`
	`127`	`+ slider.setEnabled(false);`
`129`	`128`	`return;`
`130`	`129`	`}`
`131`	`130`
`132`		`- slider.setValue(1);`
`133`	`131`	`slider.setValueFrom(1);`
	`132`	`+ slider.setValue(1);`
`134`	`133`	`slider.setValueTo(adapter.getItemCount());`
	`134`	`+ slider.setEnabled(true);`
`135`	`135`	`};`
`136`	`136`	`}`
`137`	`137`	`}`