-
Notifications
You must be signed in to change notification settings - Fork 9.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove/deprecate specifying a pageNumber directly after the hash symbol (#), to improve compatibility since other PDF viewers don't support this form (issue 7746) #7757
Remove/deprecate specifying a pageNumber directly after the hash symbol (#), to improve compatibility since other PDF viewers don't support this form (issue 7746) #7757
Conversation
} else { // Named (or explicit) destination. | ||
if ((typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) && | ||
(isPageNumber(hash) && hash < this.pagesCount)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'oh! You're obviously correct, don't know how I managed to make that kind of mistake in such a small patch. Good eye, thanks :-)
…ol (#), to improve compatibility since other PDF viewers don't support this form (issue 7746) There's no mention of our `#{pagenum}` form in http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/pdf_open_parameters.pdf, and Adobe Reader doesn't seem to support it either. Hence this patch removes support for it in the extensions, but keeps it in the `GENERIC` build with a deprecation warning and a fallback to handle it as a destination. Fixes 7746.
/botio-linux preview |
From: Bot.io (Linux)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/6a4dc89bc19e789/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/6a4dc89bc19e789/output.txt Total script time: 2.73 mins Published |
Nice work! |
…eNumber-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)
There's no mention of our
#{pagenum}
form in http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/pdf_open_parameters.pdf, and Adobe Reader doesn't seem to support it either.Hence this patch removes support for it in the extensions, but keeps it in the
GENERIC
build with a deprecation warning and a fallback to handle it as a destination.Fixes #7746.