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

Merge pull request #5953 from Snuffleupagus/issue-3498

Browse Source 
Remove no longer needed hacks that enable spacebar scrolling in Firefox (issue 3498)
This commit is contained in:
Tim van der Meij 2015-04-27 19:10:49 +02:00
commit 080ab204d6
2 changed files with 3 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
web/pdf_viewer.js
View File

@ -622,10 +622,6 @@ var PDFViewer = (function pdfViewer() {
this.container.focus();
},
blur: function () {
this.container.blur();
},
get isInPresentationMode() {
return this.presentationModeState === PresentationModeState.FULLSCREEN;
},

16
web/viewer.js
View File

@ -926,9 +926,6 @@ var PDFViewerApplication = {
// unless the viewer is embedded in a web page.
if (!self.isViewerEmbedded) {
self.pdfViewer.focus();
//#if (FIREFOX || MOZCENTRAL)
// self.pdfViewer.blur();
//#endif
}
}, function rejected(reason) {
console.error(reason);
@ -2190,16 +2187,9 @@ window.addEventListener('keydown', function keydown(evt) {
pdfViewer.focus();
}
// 32=Spacebar
if (evt.keyCode === 32 && curElementTagName !== 'BUTTON') {
//#if (FIREFOX || MOZCENTRAL)
// // Workaround for issue in Firefox, that prevents scroll keys from
// // working when elements with 'tabindex' are focused. (#3498)
// pdfViewer.blur();
//#else
if (!pdfViewer.containsElement(curElement)) {
pdfViewer.focus();
}
//#endif
if (evt.keyCode === 32 && curElementTagName !== 'BUTTON' &&
!pdfViewer.containsElement(curElement)) {
pdfViewer.focus();
}
}