Page MenuHomePhabricator

Introduce in-app previews for shared reading lists
Closed, ResolvedPublic

Assigned To
Authored By
scblr
Jan 25 2023, 1:38 PM
Referenced Files
F36859164: reading-list-share-13.png
Feb 17 2023, 6:08 PM
F36859162: Screenshot_20230217-190009.png
Feb 17 2023, 6:08 PM
F36859160: Screenshot_20230217-185739.png
Feb 17 2023, 6:08 PM
F36754754: reading-list-share-15.png
Feb 6 2023, 2:01 PM
F36754752: reading-list-share-14.png
Feb 6 2023, 2:01 PM
F36754749: reading-list-share-13.png
Feb 6 2023, 2:01 PM
F36754744: reading-list-share-12.png
Feb 6 2023, 2:01 PM
F36754737: reading-list-share-11.png
Feb 6 2023, 2:01 PM

Description

Problem (Video)
Old design
reading-list-share-12.png (1×720 px, 135 KB)
  • Participants expected to see a preview of the reading list on the web in the usability tests.
  • They asked for more information and context before a list was imported.
  • Importing something unknown felt weird to the participants.
Possible solutions
  • Since there's a legal/community constraint not to show previews on the web (yet), we're going to explore previewing a shared reading list in the app before the actual import happens (Slide)
  • We should consider renaming "Import" to "View" or "Saved"
Design (Figma)
1) Receive
reading-list-share-11.png (1×720 px, 162 KB)
2) Preview
reading-list-share-12.png (1×720 px, 824 KB)
3) Save
reading-list-share-13.png (1×720 px, 495 KB)
4) Saved
reading-list-share-14.png (1×720 px, 200 KB)
5) View
reading-list-share-15.png (1×720 px, 844 KB)

1) Receive:

  • User receives reading list via messaging app.
  • This is the journey when the app is installed. For the landing page flow, check out T327905.

2) Preview:

  • Users tap the link and are taken to this view in the app.
  • This reuses the existing reading list detail screen design but only has one call to action (Save).
  • Use a default list name structure:
    • Title: Shared list
    • Description: %datetime (e.g. Jan 31, 2023, 11:05 AM)

3) Save:

  • Users tapped the "Save" CTA, triggering a dialog. The dialog allows users to name the list and select the articles to save.
  • Hide the optional "description" field to reduce cognitive load.
  • Design does not feature a select/deselect all button to reduce cognitive load and safe screen estate
  • As the dialog contains quite some content, the keyboard is disabled, and the reading list name defaults to "Shared list %datetime". Users can tap the input field to rename it.
  • Challenge: How is scrolling handled? It needs to be explored by engineers (@cooltey featured this behavior in a prototype a while ago)

4) Saved:

  • Snackbar confirms that the reading list has been saved.
  • The "New" indicator shows that the reading list has been added.
  • Where the list is added depends on the user’s "Sort" setting.

4) View: Users can view the list as usual after the list is added.


APK: https://github.com/wikimedia/apps-android-wikipedia/pull/3829

Event Timeline

JTannerWMF triaged this task as Medium priority.Jan 31 2023, 4:02 PM

@scblr @JTannerWMF If we want this new interface to be fully translated upon release (outside the normal TWN process), we will need translations to be pre-populated in our spreadsheet. I have added a few lines at the bottom that represent the new strings added here, that will need translations.

Hi @scblr

The implementation is completed. You may still see the unchanged behaviors about the tooltips, which relate to another unmerged ticket.

Please download the APK from the ticket description, thanks!

@cooltey just checked this looks pretty good already! 👏 Two minor comments:

1) Set the default list name to Shared list %datetime to avoid alerts like this:

Screenshot_20230217-185739.png (2×1 px, 469 KB)

Reuse the title and description from here, but use a shorter form for the date and time:

Screenshot_20230217-190009.png (2×1 px, 1 MB)

2) Did you see my reply on Figma to your question?

Please use the design with the truncated title description:

reading-list-share-13.png (1×720 px, 442 KB)

@cooltey just checked this looks pretty good already! 👏 Two minor comments:

1) Set the default list name to Shared list %datetime to avoid alerts like this:

Reuse the title and description from here, but use a shorter form for the date and time:

Done. Not sure what kind of short format you'd like to see. I use MM/dd/yyyy HH:mm here.

2) Did you see my reply on Figma to your question?

Please use the design with the truncated title description:

Done.

Please download the update APK from the ticket description. Thanks!

Perfect, thanks @cooltey for the great & efficient work on this! 🥇

@scblr @JTannerWMF If we want this new interface to be fully translated upon release (outside the normal TWN process), we will need translations to be pre-populated in our spreadsheet. I have added a few lines at the bottom that represent the new strings added here, that will need translations.

The spreadsheet with the translations has been updated:

https://docs.google.com/spreadsheets/d/1gZSWZOu_IAUpY1xYB4L9511c-0w3wAJ5SNUFWSx0QYA/edit#gid=0