Skip to content
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

After upgrade to the version 10.3.3, the first page of pdf is always rendered twice, with different dimension in viewport each time #1145

Closed
oktesting opened this issue Oct 29, 2024 · 1 comment · Fixed by #1146
Milestone

Comments

@oktesting
Copy link

Bug Report or Feature Request (mark with an x)
- [x] Regression (a behavior that used to work and stopped working in a new release)
- [x] Bug report -> please search issues before submitting
- [ ] Feature request
- [ ] Documentation issue or request

Hi.
I've notice one problem with this library in this current version 10.3.3 (as far as I know)
The problem is that once the pdf is loaded, then page-rendered event is fired. In the latest version 10.3.3, the first page is always rendered twice. The first page render with incorrect width, height and scale in the page viewport. The second page render with correct ones. This behaviour causes the pdf viewer displaying first page quickly flickering as it switching from the first to second render, the page is shrink or expanded depending on its actual view width

I had reproduced and captured the problem in the library stackbliz example: https://stackblitz.com/edit/ng2-pdf-viewer

this reproduction with the library's version 10.0.0 and pdfjs-dist version 2.16.105. No issue occurred as each page is rendered once
image

this reproduction with the library's latest version 10.3.3 and pdfjs-dist version 4.6.82. The issue occurred as the first page is rendered twice
image

I hope someone could point out what seem to be the problem with this library latest version 10.3.3

Thanks to the author and all contributors for this excellent library. Cheer 🎊

@VadimDez
Copy link
Owner

Fixed in 10.3.4
thanks to @shamoon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants