Page MenuHomePhabricator

Expose the 'default list' in the Android app
Closed, ResolvedPublic

Description

Background

The reading list sync service created has a concept of a "Default" reading list, which an article can be saved to without being required to be added to a user-created reading list.
Android reading list users currently save articles to a specific user-created list; whilst iOS intends to allow both 'one-tap' quick saving which would save articles to the system 'default' list as well as the option to organize into other reading lists.

Notes:

  • The name of the Default list is an ID string in the back-end, with a localized name shown to the user (iOS initially calling it "Saved")
  • The default list cannot be renamed
  • The default list cannot be deleted
  • Default list is pinned to the top of reading lists

Problem

Until such time as Android starts making use of a default reading list, it may be confusing for users to see it appear on their device.

Proposed solution: All Android users see the default read-only bookmarks list

Pros of this approach

  • Introduces the concept of a 'default' list could be beneficial to save the step of requiring users to create an initial list
  • Users can take advantage of the (proposed) higher limit of articles for the default list
  • Showing the default list on Android is not as jarring for cross-platform users
  • Opens the door to potentially switching to the one-tap quick-saving model in future

TO DO:

PART A. Update workflow so ALL users (whether they choose to sync reading lists or not) will now see a 'Default list':
Bottom sheet when save to list is tapped
Add to reading list - no user-created lists.png (1×720 px, 229 KB)
Empty default list details screen
Default list - empty state.png (1×720 px, 79 KB)
My lists screen with overflow on default list
Default list item - overflow actions.png (1×720 px, 189 KB)
overflow on default list details screen
Default list - overflow actions.png (1×720 px, 355 KB)

Redline versions also available on the Android Zeplin board, tagged with 'Reading list sync'

Steps to QA

  1. As a new user with no user-created reading lists, tap to add an article to a reading list. It is *expected* that a bottom sheet will be shown with an option to save the article to the already existing 'Saved' default list, or to create a new reading list. (NOTE: The first-time experience is covered in the separate first time user education task T183441)
  2. Login and go to settings and opt-in to synced reading lists. It is expected that the 'Saved' default reading list is the synced as the default list by the reading list service
  3. Go to the 'My lists screen' and select the overflow of the 'Saved' default list list item. It is expected that the overflow actions do NOT include the option to delete or rename the list.
  4. Tap to view the 'Saved' reading list, and then tap on the overflow in this screen. It is also expected the overflow actions do NOT include the option to delete or rename the list.
PART B. Ensure that the Default list is with the real 'default' list when a user decides to turn on syncing

Steps to QA

  1. While NOT OPTED-IN to synced reading lists, add some articles to the default list
  2. Login and go to settings and opt-in to synced reading lists. It is expected that the 'Saved' default reading list is the synced as the default list by the reading list service
PART C. Update messaging in existing flows

Separated into sub-tasks:

PART D. Ensure no duplicate lists called 'Saved' per user

Separated into task T183588

Event Timeline

RHo renamed this task from Handling the 'default list' on Android to Expose the 'default list' in the Android app.Dec 19 2017, 6:06 PM
RHo added a project: Epic.
RHo updated the task description. (Show Details)

Change 399973 had a related patch set uploaded (by Sharvaniharan; owner: Sharvaniharan):
[apps/android/wikipedia@master] Expose default list

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

@RHo @Dbrant I feel Part B should be a separate ticket dependent on Reading list sync service integration.

@RHo @Dbrant I feel Part B should be a separate ticket dependent on Reading list sync service integration.

hi @Sharvaniharan - Happy for you to split Part B into a separate task if that makes more sense for implementation and QA

Thanks @RHo . also just now noticed the image missing next to the saved list in the first screenshot, will update the patch and fix that

Change 399973 merged by jenkins-bot:
[apps/android/wikipedia@master] Expose default list

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