From: gigaherz Date: Fri, 10 Feb 2012 16:22:06 +0000 (+0100) Subject: Changed class management to use the classList collection instead of regex. X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=b9b489c99ef45d86ebfcd041507ea5aa71983df3;p=pdf.js.git Changed class management to use the classList collection instead of regex. Fixed a bug where it wouldn't react to the first click. --- diff --git a/web/viewer.js b/web/viewer.js index 0ca0a75..e082299 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() {