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

Improve scroll experience. #147

Closed
7 tasks done
Hell0Earth opened this issue Feb 24, 2024 · 5 comments
Closed
7 tasks done

Improve scroll experience. #147

Hell0Earth opened this issue Feb 24, 2024 · 5 comments
Labels
invalid This does not seem right

Comments

@Hell0Earth
Copy link

Checklist

  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one feature request.
  • I have read and understood the contribution guidelines.
  • I optionally donated to support the Fossify mission.

Feature description

Add an option to Save loaded Thumbnails on cache or data to avoid thumbnail reload on every scroll.

Why do you want this feature?

This function will increase app storage consumption but will improve performance and UI experience for low end devices and can avoids thumbnail load delay for higher pictures.

Additional information

Great gallery 😄

@Hell0Earth Hell0Earth added feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Feb 24, 2024
@furdiburd
Copy link

Is it supposed to be cached...

@Hell0Earth
Copy link
Author

Is it supposed to be cached...

On my case, thumbnails are loading on scroll and every time that I close and open the app, loads start again.

@furdiburd
Copy link

screen-20240224-164240.mp4

worse than this?

@Hell0Earth
Copy link
Author

screen-20240224-164240.mp4

worse than this?

I'm on low end device, wait now I don't have battery, letter I will share two videos, expected behaviour and actual behaviour to show my idea/case

@Aga-C
Copy link
Member

Aga-C commented Feb 24, 2024

Thumbnails are currently cached. The slow scrolling experience is probably caused by the fade-in animation, and there is already a request for that: #136.

@Aga-C Aga-C closed this as not planned Won't fix, can't repro, duplicate, stale Feb 24, 2024
@Aga-C Aga-C added invalid This does not seem right and removed feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This does not seem right
Projects
None yet
Development

No branches or pull requests

3 participants