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

[native_pdf_view] Add Zooming without pixel loss #6

Closed
ShashankSirmour opened this issue Jun 11, 2019 · 10 comments
Closed

[native_pdf_view] Add Zooming without pixel loss #6

ShashankSirmour opened this issue Jun 11, 2019 · 10 comments
Assignees
Labels
enhancement New feature or request

Comments

@ShashankSirmour
Copy link

Package name
A name of package whom this idea concerns. Ex. native_color
zoomable image view
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I want to zoom in the rendered I tried zoomable image package but as its just image with low pixel I lose pixels
Describe the solution you'd like
A clear and concise description of what you want to happen.
I want to render a high-quality image so no issue for pixel loss or update pixel by taking an image from the zoomable image I mean just Zoomable pdf renderer
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@ShashankSirmour ShashankSirmour added the enhancement New feature or request label Jun 11, 2019
@SergeShkurko
Copy link
Member

in version 1.5.0 added crop attributes in render options

@deakjahn
Copy link
Contributor

Yes, the crop is there but the view makes no attempt at reloading the contents when the zoom changes. :-)

@deakjahn
Copy link
Contributor

deakjahn commented Apr 29, 2020

I found a possible solution, if you would like to check it out. Instead of ExtendedImage, you might want to switch to FlutterMap. Yep, I know, it seems strange at first. But, although it's conceived for maps, it could be used all right. It has full support for tiling smaller parts to show a larger picture, with pinch zoom and everything. And a FileTileProvider that you could extend to return the actual tiles of the PDF. When at zoom level 1, nothing would change but at higher zoom levels, you could generate (and cache) the smaller tiles using the crop approach.

I tried to experiment with it a little but I don't know if I can dedicate enough time to it right now. So I thought I'd mention it up front if you think you're interested to look into it. :-)

@SergeShkurko
Copy link
Member

@deakjahn ExtendedImage uses also for image switch

@deakjahn
Copy link
Contributor

No, I didn't mean that. ExtendedImage can just replace it with a larger image but if you want to allow higher zoom levels without quality loss, you simply cannot generate ten times the size, the bitmap would be too large. Using tiled images would mean that only the part the user is actually looking at gets generated.

@SergeShkurko
Copy link
Member

I understand your idea, it's just that this library is used not only for scaling, but also for turning pages. This is also quite an important factor.

@deakjahn
Copy link
Contributor

Yes, sure. I would think about a PageView, just like now, where each page is a FlutterMap. I'll try to come up with something if I have the time. :-)

@SergeShkurko
Copy link
Member

I will be glad to any help :)

@SergeShkurko SergeShkurko changed the title Add Zooming without pixel loss [native_pdf_view] Add Zooming without pixel loss Aug 9, 2020
@SergeShkurko SergeShkurko pinned this issue Nov 4, 2021
@SergeShkurko
Copy link
Member

SergeShkurko commented Nov 4, 2021

I start work on this issue

see #238

@SergeShkurko
Copy link
Member

added in pdfx

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

No branches or pull requests

3 participants