Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ensure that Scroll/Spread mode buttons are correctly reset, when the document is closed

Browse Source 
Since the Scroll/Spread modes are now document specific, as all other properties such as page/scale/rotation, ensure that the toolbar is always correctly reset.
This commit is contained in:
Jonas Jenwald 2018-06-30 14:54:33 +02:00
parent bb193dc501
commit ff26d419dd
1 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
web/secondary_toolbar.js
View File

@ -140,6 +140,10 @@ class SecondaryToolbar {
this.pageNumber = 0; this.pageNumber = 0;
this.pagesCount = 0; this.pagesCount = 0;
this._updateUIState(); this._updateUIState();
// Reset the Scroll/Spread buttons too, since they're document specific.
this.eventBus.dispatch('resetscrollmode', { source: this, });
this.eventBus.dispatch('resetspreadmode', { source: this, });
} }
_updateUIState() { _updateUIState() {
@ -189,7 +193,7 @@ class SecondaryToolbar {
} }
_bindScrollModeListener(buttons) { _bindScrollModeListener(buttons) {
this.eventBus.on('scrollmodechanged', function(evt) { function scrollModeChanged(evt) {
buttons.scrollVerticalButton.classList.remove('toggled'); buttons.scrollVerticalButton.classList.remove('toggled');
buttons.scrollHorizontalButton.classList.remove('toggled'); buttons.scrollHorizontalButton.classList.remove('toggled');
buttons.scrollWrappedButton.classList.remove('toggled'); buttons.scrollWrappedButton.classList.remove('toggled');
@ -205,11 +209,18 @@ class SecondaryToolbar {
buttons.scrollWrappedButton.classList.add('toggled'); buttons.scrollWrappedButton.classList.add('toggled');
break; break;
} }
}
this.eventBus.on('scrollmodechanged', scrollModeChanged);
this.eventBus.on('resetscrollmode', (evt) => {
if (evt.source === this) {
scrollModeChanged({ mode: ScrollMode.VERTICAL, });
}
}); });
} }
_bindSpreadModeListener(buttons) { _bindSpreadModeListener(buttons) {
this.eventBus.on('spreadmodechanged', function(evt) { function spreadModeChanged(evt) {
buttons.spreadNoneButton.classList.remove('toggled'); buttons.spreadNoneButton.classList.remove('toggled');
buttons.spreadOddButton.classList.remove('toggled'); buttons.spreadOddButton.classList.remove('toggled');
buttons.spreadEvenButton.classList.remove('toggled'); buttons.spreadEvenButton.classList.remove('toggled');
@ -225,6 +236,13 @@ class SecondaryToolbar {
buttons.spreadEvenButton.classList.add('toggled'); buttons.spreadEvenButton.classList.add('toggled');
break; break;
} }
}
this.eventBus.on('spreadmodechanged', spreadModeChanged);
this.eventBus.on('resetspreadmode', (evt) => {
if (evt.source === this) {
spreadModeChanged({ mode: SpreadMode.NONE, });
}
}); });
} }