Page MenuHomePhabricator

Integrate lazy image loading on Android (lazy load images)
Closed, ResolvedPublic2 Estimated Story Points

Description

Observe the logcat and see that section images are only fetched when scrolling through those sections.

Event Timeline

Niedzielski renamed this task from Integrate lazy image loading on Android to Integrate lazy image loading on Android (lazy load images).May 5 2017, 7:21 PM

Change 362128 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] WIP: New: lazy image loading

https://gerrit.wikimedia.org/r/362128

Change 362129 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[mediawiki/extensions/MobileApp@master] Update: add lazy image loading placeholder style

https://gerrit.wikimedia.org/r/362129

Change 362129 abandoned by Niedzielski:
WIP: Update: add lazy image loading placeholder style

Reason:
Moved to page-library

https://gerrit.wikimedia.org/r/362129

Change 363504 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] Hygiene: upgrade wikimedia-page-library to v4.0.1

https://gerrit.wikimedia.org/r/363504

Change 363505 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] Update: add / remove dark mode class from root element

https://gerrit.wikimedia.org/r/363505

Change 363504 merged by jenkins-bot:
[apps/android/wikipedia@master] Hygiene: upgrade wikimedia-page-library to v4.0.1

https://gerrit.wikimedia.org/r/363504

Change 363505 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: add / remove dark mode class from root element

https://gerrit.wikimedia.org/r/363505

Change 362128 merged by jenkins-bot:
[apps/android/wikipedia@master] New: lazily load images

https://gerrit.wikimedia.org/r/362128

Hi, I haven't worked with logcat and there are no Testing Criteria steps for recreating the scenario that led to the issue, I will let someone else maybe @ABorbaWMF do this one instead.

Tested on a Nexus 4 with Android 5.1 and a Pixel with Android 7.1.1

@Nicholas.tsg I think some general viewing of images is what we need to do. I am not seeing any missing images. I used a few articles like 'Berlin" that are image heavy.