From b9b489c99ef45d86ebfcd041507ea5aa71983df3 Mon Sep 17 00:00:00 2001 From: gigaherz Date: Fri, 10 Feb 2012 17:22:06 +0100 Subject: [PATCH] Changed class management to use the classList collection instead of regex. Fixed a bug where it wouldn't react to the first click. --- web/viewer.js | 27 ++++++++++----------------- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/web/viewer.js b/web/viewer.js index 0ca0a753d..e0822990b 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -576,25 +576,18 @@ var PDFView = { var sidebar = document.getElementById('sidebar'); var pinIcon = document.getElementById('pinIcon'); - sidebar.className = sidebar.className.replace( - /(?:^|\s)released(?!\S)/ , - ''); - sidebar.className = sidebar.className.replace( - /(?:^|\s)pinned(?!\S)/ , - ''); - pinIcon.className = pinIcon.className.replace( - /(?:^|\s)released(?!\S)/ , - ''); - pinIcon.className = pinIcon.className.replace( - /(?:^|\s)pinned(?!\S)/ , - ''); - - var newClass = this.pinState ? ' pinned' : ' released'; - - sidebar.className += newClass; - pinIcon.className += newClass; + sidebar.classList.remove('released'); + pinIcon.classList.remove('released'); + sidebar.classList.remove('pinned'); + pinIcon.classList.remove('pinned'); this.pinState = !this.pinState; + + var newClass = this.pinState ? 'pinned' : 'released'; + + sidebar.classList.add(newClass); + pinIcon.classList.add(newClass); + }, getVisiblePages: function pdfViewGetVisiblePages() {