Browse Source

Merge pull request #7757 from Snuffleupagus/setHash-remove-pageNumber-directly-after-hash

Remove/deprecate specifying a pageNumber directly after the hash symbol (#), to improve compatibility since other PDF viewers don't support this form (issue 7746)
Tim van der Meij 9 years ago committed by GitHub
parent
commit
1c3fb175dd
  1. 10
      web/pdf_link_service.js

10
web/pdf_link_service.js

@ -270,9 +270,15 @@ var PDFLinkService = (function PDFLinkServiceClosure() { @@ -270,9 +270,15 @@ var PDFLinkService = (function PDFLinkServiceClosure() {
mode: params.pagemode
});
}
} else if (isPageNumber(hash)) { // Page number.
this.page = hash | 0;
} else { // Named (or explicit) destination.
if ((typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) &&
isPageNumber(hash) && hash <= this.pagesCount) {
console.warn('PDFLinkService_setHash: specifying a page number ' +
'directly after the hash symbol (#) is deprecated, ' +
'please use the "#page=' + hash + '" form instead.');
this.page = hash | 0;
}
dest = unescape(hash);
try {
dest = JSON.parse(dest);

Loading…
Cancel
Save