-
Notifications
You must be signed in to change notification settings - Fork 32
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
Keep ZoomState and position if loading Images in same screen. #104
Comments
Is the overall goal here to retain the pan and zoom state across image changes? If so, I'd definitely like to support it but it has proven to be non-trivial in my past attempts. For your thumbnails, have you tried using coil's placeholder for displaying them? Telephoto will smoothly swap out placeholders when their full quality versions are ready and even block zoom events until then. https://saket.github.io/telephoto/zoomableimage/#placeholders For smoothly transitioning from a preview image to its full-sized image, is the image resolution known before hand for the full-sized variant? |
Hi saket, thanks for you reply.
In my use case, it is essential to provide good UX. We don't want to reset the current zoom state as soon the fullSize downloaded. If you see my video, each quality transition: thumbnail -> preview -> fullSize, the central position is changing. I guess it is caused by the different size of picture (thumbnail/preview/fullsize). But I have no idea to handle it.
Great to know!
Unfortunately, we don't know. Again, thank you so much. |
Yeah it'd be great @saket if Telephoto can support it. |
Gotcha. Yea I wish Coil had support for dynamic thumbnails.
Noted. In theory it should be possible to retain the zoom and offset values if the new image shares the same aspect ratio as the old one. I'll try working on this soon. |
Thank you very much. |
Hi Saket, any progress on this? |
I unfortunately haven't been able to find time to work on this yet. I made some progress in the last release by adapting zoom & pan values to a new viewport size: Lines 15 to 24 in edd9974
I think we can write something similar for adapting to new content sizes. Wanna help me out? |
Hello, my job has changed and I am unable to support the work. I will notify my colleagues for follow-up. Thank you very much. |
Hello my situation that I have ImagePreview pager and I will show thumbnail first then Preview then fullSizeImage (3 different paths), which require 3 downloading process in same screen. I am wondering whether it is possible to support
Now my code like this
I know it may be not easy, so could you please guide if I can custom this logic in somewhere? Thank you very much.
flash.and.position.move.mp4
The text was updated successfully, but these errors were encountered: