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 0ca0a753d9eeef849f71eaa760c23ec4034723bf..e0822990b4e36525afcd57de421a21c211be2750 100644 (file)
--- 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() {