Page MenuHomePhabricator

Reading list walk through tweaks and updates
Closed, ResolvedPublic

Description

Spreadsheet

https://docs.google.com/spreadsheets/d/1ycWnt2IuZSgC2zNxGXjaFvqxkaC0M2eO1ka_Kv9BN40/edit?usp=sharing


Interaction updates

Updates/bugs with interactions in the Reading list feature

ScreenSteps to reproduceExpected behaviorActual behaviorProposed solutionRelated links
Create a new listType a reading list name in the reading list name field then hit the return keyCursor is moved to the Description fieldA second row is added below the titleChange the 'return' key to a blue 'next' key, tapping on next moves to the following text field. The key will turn to a 'done' key after the description has been entered, tapping on the 'done' key creates a new reading list
Create a new listType a reading list name in the reading list name fieldAn 'x' (clear field) button appears when the text field is active.There is no clear field button when the text field is activeAdd a clear field button to the text field when it is active, tapping on this button clears entered text
Saved articles tabTap on the sort linkThe current sort is shown as selected with a check markAll options are shown without a check markAdd a checkmark after the selected sorthttps://zpl.io/aXKJBkj
Saved articles tabSave 6+ articles then scroll to down the list of saved articlesScrolling down the list of saved articles hides the Search field and Sort link, scrolling back to the top of the list reveals them.Search field persistsUtilize a similar 'show/hide' behavior with the search bar in the saved articles tab as on the Explore feedhttps://zpl.io/aBwGMqQ and https://zpl.io/blrx5Ze
Saved articles tabSave an article to a reading list, then tap on the reading list tagOpens the associated listOpens the article Tapping on the tab should open the associated list, tapping elsewhere in the cell opens the articlehttps://phabricator.wikimedia.org/T177546
Bulk editingTap on the edit link on the Saved articles or Reading lists viewBulk editing toolbar is visibleBulk editing toolbar is only visible after a selection has been made Make the toolbar visible when the bulk editing view is activated, which should help users to have a better understanding of what actions they can takehttps://zpl.io/bJQxy73
Article viewTap on the 'Save' button in the article toolbarArticle is saved and a hint to add the article to a reading list is shown at the bottom of the screenArticle is savedShow the hint that is used on the Explore feed to add the article to a reading list on the article viewhttps://zpl.io/V4EMWeN
3D PeekPeek an article and tap on Save for laterPeek closes, article is saved and hint to add the article to a reading list is shown at the bottom of the screenPeek closes, article is savedShow the hint that is used on the Explore feed to add the article to a reading list on the screen that is shown after the peek is closedhttps://zpl.io/V4EMWeN
Table viewSwipe left on an article cell, then tap on the 'save' iconArticle is saved and a hint to add the article to a reading list is shown at the bottom of the screenArticle is savedShow the hint that is used on the Explore feed to add the article to a reading list on the table viewhttps://zpl.io/V4EMWeN

Visual tweaks

Visual changes and updates for the Reading list feature

ScreenProposed updateRelated link
Create a new listThe design as currently implemented uses a slightly compacted version of the login flow form. The proposal is to utilize the same layout as the login form. Left and right margins should be 15px, form fields should include relevant hint texthttps://zpl.io/VQM7wvn
Saved articles viewSaved articles' and 'Reading lists' should be the same font size as article titles (16pt)https://zpl.io/aBwGMqQ
Saved articles viewHR should appear under the 'Saved articles' and 'Reading lists' selector (.5pt, #d3d6da)https://zpl.io/aBwGMqQ
Saved articles viewEdit and Sort links should use the same font size (16pt) and color (#36c). Both links should be right aligned with a 20pt right marginhttps://zpl.io/aBwGMqQ
Bulk editingIn the bulk editing view, selection indicators should utilize an unfilled circle for the unselected state and a filled blue circle with a checkmark for the selected state. Selected cells should use a background fill of #eaf3ffhttps://zpl.io/bJQxy73
Bulk editingThe bulk editing actions toolbar should be the same height as the app tool bar (currently it appears to be a pixel or two taller)https://zpl.io/bJQxy73
Bulk editingThe bulk editing actions toolbar should have a fill color of #f8f9fahttps://zpl.io/bJQxy73
Add to listThe padding above the title should be 15pthttps://zpl.io/VDpwyAJ
Add to listA HR should appear between reading list cells. The HR is left aligned with 55pt of right padding. The HR is .5pt thick and #d3d6dahttps://zpl.io/VDpwyAJ
Add to listInclude thumbnails for lists that have one or more articles with a featured imagehttps://zpl.io/VDpwyAJ
Add to listInclude alphabetical list headershttps://zpl.io/VDpwyAJ
Add to listRemove subtitle from all lists except for the default listhttps://zpl.io/VDpwyAJ
Reading lists viewA HR should appear between reading list cells. The HR is left aligned with 55pt of right padding. The HR is .5pt thick and #d3d6dahttps://zpl.io/2y10nqJ
Reading lists viewInclude the number of articles in the list above the list namehttps://zpl.io/2y10nqJ
Reading lists viewInclude thumbnails for lists that have one or more articles with a featured imagehttps://zpl.io/2y10nqJ

Event Timeline

Looks like the '0 articles' label is appearing behind the 'Saved' label:

Screen Shot 2018-01-31 at 11.42.01 AM.png (1×2 px, 563 KB)

Noticed it was barely peeking out of the right side.

@cmadeo
One visual tweak to consider - when only showing a single cell we may want to hide the horizontal divider lines:

Screen Shot 2018-01-31 at 12.06.02 PM.png (1×752 px, 152 KB)

We may also want to always hide the top divider of the top cell.

Looks like the '0 articles' label is appearing behind the 'Saved' label:

Screen Shot 2018-01-31 at 11.42.01 AM.png (1×2 px, 563 KB)

Noticed it was barely peeking out of the right side.

@Mhurd we're actually going to remove the default list from this view.

@cmadeo

Guessing we'll probably want to trim out leading and trailing spaces when saving list names/descriptions:

Screen Shot 2018-01-31 at 12.10.27 PM.png (1×862 px, 89 KB)

Screen Shot 2018-01-31 at 12.10.34 PM.png (1×862 px, 111 KB)

JMinor triaged this task as Medium priority.
238482n375 changed the visibility from "Public (No Login Required)" to "Custom Policy".
This comment was removed by Reedy.
Aklapper changed the visibility from "Custom Policy" to "Public (No Login Required)".