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

Fast swiping through images does not work #10

Closed
guettli opened this issue Sep 12, 2015 · 4 comments
Closed

Fast swiping through images does not work #10

guettli opened this issue Sep 12, 2015 · 4 comments

Comments

@guettli
Copy link

guettli commented Sep 12, 2015

If you swipe through images fast, the swiping gets ignored often.

Only slow swiping seems to work.

My device: Samsung Tablet SM-T800

@guettli
Copy link
Author

guettli commented Sep 12, 2015

How fast you can swipe seems to related to the image size.

Big pictures from my Canon 40D (jpg) can't be swiped fast. But small images from my mobile phone can be swiped fast.

@k3b
Copy link
Owner

k3b commented Sep 12, 2015

In current implementation swiping pauses until loading the next image into memory has finished. The bigger the image the more time is needed :-( . I have only small images on my tablet so i was not aware of the problem.

I am not shure if i can fix this ...

@k3b k3b added this to the v0.4.2 milestone Sep 12, 2015
k3b added a commit that referenced this issue Sep 13, 2015
…loading: initially the view is loaded with low res image. on first zoom it is reloaded with maximum resolution
@k3b
Copy link
Owner

k3b commented Sep 13, 2015

I made swiping faster by initially loading a low res image. on first zoom it is reloaded with maximum resolution.

will be available in v0.4.2

@k3b k3b closed this as completed Sep 13, 2015
@guettli
Copy link
Author

guettli commented Sep 14, 2015

Thank you for the fast response.

k3b added a commit to k3b/PhotoView that referenced this issue Jun 13, 2016
… fix memory leak

This is a Backport of PhotoView improvements from  https://github.com/k3b/APhotoManager

* Faster initial loading so swiping through ViewPager is more fluent:
  * initially the view is loaded with low res image. (defined via PhotoView.setImageXxx)
  * on first zoom it is reloaded (defined via PhotoView.setImageReloadFile())
  * see k3b/APhotoManager#10 Fast swiping through images does not work
* Fix: No crash when loading images bigger than 4096x4096 or out of memory
  * HugeImageLoader.java
  * see k3b/APhotoManager#15
* Improved logging
  * possibility to reduce logging: either logg all or info+warn+err or warn+err
    * via LogManager.setLogger() and LogManager.enableDebug()
  * many internal functions get a string parameter `why` so logging shows why a function was called
* fix memory leak
k3b added a commit to k3b/PhotoView that referenced this issue Jun 13, 2016
… fix memory leak

This is a Backport of PhotoView improvements from  https://github.com/k3b/APhotoManager

* Faster initial loading so swiping through ViewPager is more fluent:
  * initially the view is loaded with low res image. (defined via PhotoView.setImageXxx)
  * on first zoom it is reloaded (defined via PhotoView.setImageReloadFile())
  * see k3b/APhotoManager#10 Fast swiping through images does not work
* Fix: No crash when loading images bigger than 4096x4096 or out of memory
  * HugeImageLoader.java
  * see k3b/APhotoManager#15
* Improved logging
  * possibility to reduce logging: either logg all or info+warn+err or warn+err
    * via LogManager.setLogger() and LogManager.enableDebug()
  * many internal functions get a string parameter `why` so logging shows why a function was called
* fix memory leak
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants