Page MenuHomePhabricator

As a user of reading lists, I would like to have increased offline support
Closed, DeclinedPublic

Assigned To
None
Authored By
cmadeo
Oct 5 2017, 11:06 PM
Referenced Files
F10798249: Offline - Reading list.png
Nov 15 2017, 1:04 AM
F10798250: Offline - Queue article download .png
Nov 15 2017, 1:04 AM
F10797859: Offline - Top of article.png
Nov 15 2017, 12:51 AM
F10797858: Offline - Article footer.png
Nov 15 2017, 12:51 AM
F10797857: Offline - Queue article download .png
Nov 15 2017, 12:51 AM

Description

Why are we doing this?

Android currently supports a robust set of offline features including the ability for users to save whole reading lists or individual articles offline. Android also includes messaging alerting users that they are reading a saved version of an article when they are offline and when a user clicks into a link that is not saved they are shown an option to queue that article to be saved when they are next online.

Suggested features

  • Messaging alerting a user that they are offline when they are reading a saved article, which includes the timestamp of last update
  • Messaging when a user clicks on an article that they do not currently have saved to queue this article to be saved when they are back online
NOTE: Articles that have been added to a reading list but not yet downloaded are badged on the Saved tab, which is defined in the following ticket T187968

Designs

Mocks and Zeplin links

Offline toast messageQueue article to sync later
Offline - Top of article.png (667×375 px, 174 KB)
}
Offline - Queue article download .png (667×375 px, 47 KB)
Zeplin: https://zpl.io/a8ElxY6Zeplin: https://zpl.io/V1MyjkQ

Design details

  • Toast message only presented when user is offline
  • Tapping on the 'Add to reading list' CTA on the queue article download message opens the 'Add to reading list' overlay

Copy

TypeWhen ShownCopy
AlertWhen a user accesses a downloaded article when offlineYou are reading a version of this article from [Date]

Scenario

Pre-requisite: User is offline with the article 'Ching Shih' saved

  1. User goes to their Saved article list
  2. User taps on the article 'Ching Shih' which they had previously saved while online and which is downloaded to their device
  3. Alert is shown informing user that they are offline and accessing an old version of the article (https://zpl.io/a8ElxY6)
  4. User taps on a link in the article to the article 'Cantonese,' which they do not have downloaded on their device
  5. The 'Article cannot be loaded' model is shown (https://zpl.io/V1MyjkQ)
  6. User taps on the CTA to 'Add to reading list'
  7. User adds 'Cantonese' to their reading list 'Read later'
  8. Model is closed and confirmation hint is shown confirming that 'Cantonese has been added to Read later'

End result: Cantonese has been queued for download and added to the user's list 'Read later'

Event Timeline

cmadeo renamed this task from As a to As a user of reading lists, I would like to have increased offline support .Oct 5 2017, 11:06 PM
cmadeo created this task.

@cmadeo
The second image footer strings says Wikipedia English - should we flip those around?

@Mhurd, Also I need to update the last image. This ticket is pre T187968, so we should use the yellow dot rather than the text that is shown in the last mock

@cmadeo oh one more question - it looks like this ticket covers the article view when offline and trying to add a link's article to a list, but we don't have the article view configured to show the hint yet when the user is not offline (peek a link, select "Save for later", no hint presently showing)... should that be hooked up first? I'm thinking wiring the non-offline handling first may be the way to go...

@Mhurd agreed, starting with the offline hint makes the most sense.

@cmadeo Confirmed loading an already saved article when online does fetch the latest revision number and compare it with saved revision and fetches latest if the revision id doesn't match the saved one. So per your question it looks like we're ok to remove the second screenshot.

@JMinor said park this in blocked or waiting pending further chat.

cmadeo updated the task description. (Show Details)

@Mhurd just an FYI that I've updated this ticket based on our check-in

Issue for further discussion:

  • there are many places which can be tapped to trigger an article to be loaded (article links, from saved/history lists, search results, feed cards, places, etc)
  • decide whether we want the "Article cannot be loaded" panel to appear *before* attempting to push the article view controller vs showing it over the article view controller after it appears
  • or instead, as @cmadeo suggested, perhaps incorporate it into the article "empty view" instead of showing a panel - easier to implement and works regardless of how the article was loaded (from saved, history, feed, link tap etc.)

@cmadeo thanks!

Reminder that we'll probably need to update the description Scenario too after you decide specifics. (I'm leaning toward your idea of incorporating the panel bits into the article empty state...)

Mhurd removed Mhurd as the assignee of this task.Mar 18 2020, 7:34 PM
Mhurd subscribed.