Fix for amp-base-carousel next arrow not hidden when visibleCount > 1

ampproject · Sep 8, 2020 · c7506dc · c7506dc
1 parent 63848d3
commit c7506dc
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/extensions/amp-base-carousel/0.1/amp-base-carousel.js b/extensions/amp-base-carousel/0.1/amp-base-carousel.js
@@ -537,14 +537,15 @@ class AmpCarousel extends AMP.BaseElement {
   updateUi_() {
     const index = this.carousel_.getCurrentIndex();
     const loop = this.carousel_.isLooping();
+    const visibleCount = this.carousel_.getVisibleCount();
     // TODO(sparhami) for Shadow DOM, we will need to get the assigned nodes
     // instead.
     iterateCursor(this.prevArrowSlot_.children, (child) => {
       const disabled = !loop && index === 0;
       toggleAttribute(child, 'disabled', disabled);
     });
     iterateCursor(this.nextArrowSlot_.children, (child) => {
-      const disabled = !loop && index === this.slides_.length - 1;
+      const disabled = !loop && index >= this.slides_.length - visibleCount;
       toggleAttribute(child, 'disabled', disabled);
     });
     toggleAttribute(