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

Merge pull request #10897 from Snuffleupagus/rm-isPageVisible-check

Browse Source 
Remove a superfluous `linkService.isPageVisible` check from `PDFFindController` (PR 10217 follow-up)
This commit is contained in:
Tim van der Meij 2019-06-11 23:40:29 +02:00 committed by GitHub
commit 64b520404e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
web/pdf_find_controller.js
View File

@ -252,9 +252,9 @@ class PDFFindController {
// over matches at the top/bottom of pages thus making them completely // over matches at the top/bottom of pages thus making them completely
// inaccessible when there's multiple pages visible in the viewer. // inaccessible when there's multiple pages visible in the viewer.
if (pageNumber >= 1 && pageNumber <= linkService.pagesCount && if (pageNumber >= 1 && pageNumber <= linkService.pagesCount &&
linkService.page !== pageNumber && linkService.isPageVisible && pageNumber !== linkService.page &&
!linkService.isPageVisible(pageNumber)) { !linkService.isPageVisible(pageNumber)) {
break; return true;
} }
return false; return false;
case 'findhighlightallchange': case 'findhighlightallchange':