From ca1dbda3483d32396c86417c62f129c4c87553db Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Tue, 12 Aug 2014 12:34:27 +0200 Subject: [PATCH] Disable the "Go to First/Last Page" buttons when the first/last page is currently active Since the "Previous/Next Page" buttons are disabled when the first/last page is active, I think that for the sake of consistency the "Go to First/Last Page" should behave the same way. --- web/viewer.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/web/viewer.js b/web/viewer.js index 86f17b52c..49d79688e 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -2211,8 +2211,13 @@ window.addEventListener('pagechange', function pagechange(evt) { } } } + var numPages = PDFView.pages.length; + document.getElementById('previous').disabled = (page <= 1); - document.getElementById('next').disabled = (page >= PDFView.pages.length); + document.getElementById('next').disabled = (page >= numPages); + + document.getElementById('firstPage').disabled = (page <= 1); + document.getElementById('lastPage').disabled = (page >= numPages); }, true); function handleMouseWheel(evt) {