From 7af46b433d576bf8f38142d56c74e3e63284ec1a Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 24 Sep 2014 10:35:01 +0200 Subject: [PATCH] Fix a regression preventing page change in the Stepper in the debugger This is a regression from https://github.com/mozilla/pdf.js/commit/ddd3c8fc2fe7b8715bed6d6f146c206d133b5dd6, which prevents changing pages in the Stepper. (Apart from fixing the bug, for the sake of consistency, I also removed one instance of `parseInt`.) --- web/debugger.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/web/debugger.js b/web/debugger.js index 2251682d8..81da232ee 100644 --- a/web/debugger.js +++ b/web/debugger.js @@ -211,6 +211,7 @@ var StepperManager = (function StepperManagerClosure() { }, selectStepper: function selectStepper(pageIndex, selectPanel) { var i; + pageIndex = pageIndex | 0; if (selectPanel) { this.manager.selectPanel(this); } @@ -419,7 +420,7 @@ var Stepper = (function StepperClosure() { var allRows = this.panel.getElementsByClassName('line'); for (var x = 0, xx = allRows.length; x < xx; ++x) { var row = allRows[x]; - if (parseInt(row.dataset.idx, 10) === idx) { + if ((row.dataset.idx | 0) === idx) { row.style.backgroundColor = 'rgb(251,250,207)'; row.scrollIntoView(); } else {