[SPIKE] Optimize performance when importing a shared reading list
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	scblr
	Jan 25 2023, 1:58 PM

Description

Problem

Importing a reading list took a very long time. In this example video, importing six articles took ~ 30 seconds; this was the case in most tests.
Reading list header images took a long time to load. After the list had been imported, the article images in the header took a long time to be displayed in higher quality. This happened even when participants had a good internet connection (Video)

Solution

Investigate how the performance can be optimized when importing a shared reading list
Investigate how to optimize the image loading behavior when importing lists

Related Objects
Search...

Status	Assigned	Task
Open	None	T313269 Shareable Reading Lists
Open	None	T327879 Optimize 'Share reading lists' per feedback from usability tests
Resolved	cooltey	T327907 [SPIKE] Optimize performance when importing a shared reading list

Event Timeline

scblr created this task.Jan 25 2023, 1:58 PM

scblr mentioned this in T327879: Optimize 'Share reading lists' per feedback from usability tests.Jan 25 2023, 2:07 PM

scblr updated the task description. (Show Details)Jan 25 2023, 2:13 PM

LGoto triaged this task as Low priority.Jan 31 2023, 5:36 PM

LGoto edited projects, added Wikipedia-Android-App-Backlog (Android Release FY2022-23); removed Wikipedia-Android-App-Backlog.

LGoto moved this task from Epics in Progress to Ready for dev on the Wikipedia-Android-App-Backlog (Android Release FY2022-23) board.

Per discussion:

This relates to the T327909 where the current flow shows the "imported" list only after the articles are downloaded completely, which will also affect the issue that displays the size (%d MB) of the reading lists.

cooltey claimed this task.Feb 3 2023, 11:13 PM

cooltey moved this task from Ready for dev to Doing on the Wikipedia-Android-App-Backlog (Android Release FY2022-23) board.

cooltey moved this task from Doing to Code Review on the Wikipedia-Android-App-Backlog (Android Release FY2022-23) board.Feb 8 2023, 1:00 AM

The research is completed. I have a PR of fixing it and also fixing the zero MB size when importing a list.

See demo: @scblr Since there's no alpha APK available for testing, here is a demo video for you to check:
https://youtube.com/shorts/sg2wYTH0xIE

Looks good, @cooltey. Minor feedback: can you hide the indicator completely when it’s = 0.00 MB (at the beginning)? Start to show the indicator once it’s bigger than 0.00 MB.

Hi @scblr. The fix has been handled in this ticket T328474 and now you can see it on the latest Alpha.

Dbrant moved this task from Merged and Waiting to QA signoff on the Wikipedia-Android-App-Backlog (Android Release FY2022-23) board.Feb 13 2023, 9:19 PM

Testing on list of various sizes it appears to be faster now. The 6 article list I used for testing took ~17 seconds on a decent network. Looks good on 2.7.50429-r-2023-02-13

Great job

JTannerWMF moved this task from Ready for PM signoff to Ready for release on the Wikipedia-Android-App-Backlog (Android Release FY2022-23) board.Feb 21 2023, 11:44 PM

Dbrant closed this task as Resolved.Thu, Feb 23, 4:07 PM

[SPIKE] Optimize performance when importing a shared reading listClosed, ResolvedPublicActions