No need to be paranoid, the code isn't going to contain both classes at once unless someone messes up, and then it's not my fault.
This commit is contained in:
parent
b9b489c99e
commit
8a22a43cf3
@ -576,18 +576,20 @@ var PDFView = {
|
||||
var sidebar = document.getElementById('sidebar');
|
||||
var pinIcon = document.getElementById('pinIcon');
|
||||
|
||||
sidebar.classList.remove('released');
|
||||
pinIcon.classList.remove('released');
|
||||
sidebar.classList.remove('pinned');
|
||||
pinIcon.classList.remove('pinned');
|
||||
var oldClass = 'released';
|
||||
var newClass = 'pinned';
|
||||
|
||||
this.pinState = !this.pinState;
|
||||
|
||||
var newClass = this.pinState ? 'pinned' : 'released';
|
||||
if (this.pinState) {
|
||||
oldClass = 'pinned';
|
||||
newClass = 'released';
|
||||
}
|
||||
|
||||
sidebar.classList.remove(oldClass);
|
||||
pinIcon.classList.remove(oldClass);
|
||||
sidebar.classList.add(newClass);
|
||||
pinIcon.classList.add(newClass);
|
||||
|
||||
this.pinState = !this.pinState;
|
||||
},
|
||||
|
||||
getVisiblePages: function pdfViewGetVisiblePages() {
|
||||
|
Loading…
x
Reference in New Issue
Block a user