Bug 809782 - remove background-image for offscreen thumbnails #6897

Merged
merged 1 commit into from Dec 10, 2012

Conversation

Projects
None yet
2 participants

This pull request dramatically decreases gallery memory usage when there are many images in the gallery by no setting the background-image style of thumbnail elements that are not visible. If there are 1000 images on the phone, I expect this patch to make the gallery process 40mb leaner.

The changes to the gallery app itself are small. The interesting code is in shared/js/visibility_monitor.js which tracks the scrolling list of thumbnails to and calls callbacks when they go on and off screen.

Member

daleharvey commented Dec 10, 2012

Looks great, cheers david

daleharvey merged commit 6cd734d into mozilla-b2g:master Dec 10, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment