Change progress bar to not handle active page, since active page upda…

…tes progress bar through PAGE_PROGRESS events
3qnexx · Oct 19, 2017 · 0500899 · 0500899
1 parent d19c061
commit 0500899
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/extensions/amp-story/0.1/progress-bar.js b/extensions/amp-story/0.1/progress-bar.js
@@ -109,10 +109,13 @@ export class ProgressBar {
   setActivePageIndex(pageIndex) {
     this.assertValidPageIndex_(pageIndex);
     for (let i = 0; i < this.pageCount_; i++) {
-      if (i <= pageIndex) {
+      if (i < pageIndex) {
         this.updateProgress(i, 1.0);
-      } else {
+      } else if (i > pageIndex) {
         this.updateProgress(i, 0.0);
+      } else {
+        // The active page manages its own progress by firing PAGE_PROGRESS
+        // events to amp-story.  As such, its progress is not set here.
       }
     }
   }