akira/pdfjs-dist - pdfjs-dist - Gitea: Git with a cup of tea

Tree: 29947b5a57

Author	SHA1	Message	Date
Jonas Jenwald	b7cb44af88	Remove the `previousPageNumber` parameter from the `pagechanging`/pagechange` events, and stop dispatching the events if the input is out of bounds This patch attempts to cleanup a couple of things: - Remove the `previousPageNumber` paramater. Prior to PR 7289, when the events were dispatched even when the active page didn't change, it made sense to be able to detect that in an event listener. However, now that's no longer the case, and furthermore other similar events (e.g. `scalechanging`/`scalechange`) don't include information about the previous state. - Don't dispatch the events when the value passed to `set currentPageNumber` is out of bounds. Given that the active page doesn't change in this case, again similar to PR 7289, I don't think that the events should actually be dispatched in this case. - Ensure that the value passed to `set currentPageNumber` is actually an integer, to avoid any issues (note how e.g. `set currentScale` has similar validation code). Given that these changes could possibly affect the PDF.js `mochitest` integration tests in mozilla-central, in particular https://dxr.mozilla.org/mozilla-central/source/browser/extensions/pdfjs/test/browser_pdfjs_navigation.js, I ran the tests locally with this patch applied to ensure that they still pass.	9 years ago
jazzy-em	0a347ec04d	Added multiple term search functionality (with default phrase search)	9 years ago
Yury Delendik	9f68e3f1bc	Disable pagechange event if page is not changing.	9 years ago
Yury Delendik	3e6e294fd4	Refactors PDFFindBar and FirefoxCom find events.	9 years ago
Yury Delendik	7fd3db9977	Adds EventBus.	9 years ago