espresso3389 / pdfrx

pdfrx is yet another PDF viewer implementation that built on the top of pdfium. The plugin currently supports Android, iOS, Windows, macOS, Linux, and Web.
MIT License
60 stars 36 forks source link

Fix RSS exploding when zoom in on large pages #122

Closed yhyh0 closed 1 month ago

yhyh0 commented 1 month ago

Might also worth considering that we crop the scaled image to cover only the visible rect... Any thoughts? Would that break too many parts?

Now it takes about 120mb for a 3000*4000 page to scale up to 2.0, it won't work well if we scale it all the way up to the max scale of 7 or higher

yhyh0 commented 1 month ago

@espresso3389 closed the pr since I saw your changes, but the question remains.. Still value your thoughts when you get a chance to look at it. And I notice a few blink(no pdf image showing up for a ms) when move/scale using the latest 1.0.52. I'm looking at it, but if you already know/fix the issue, please let me know :)

espresso3389 commented 1 month ago

Yes, I also realized there are blinks on certain operation. I'm checking it.