Changed class management to use the classList collection instead of regex.
Fixed a bug where it wouldn't react to the first click.
This commit is contained in:
parent
24b72f70bf
commit
b9b489c99e
@ -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() {
|
||||
|
Loading…
x
Reference in New Issue
Block a user