Page MenuHomePhabricator

As a new user I should be able to remove languages while I'm on-boarding the app
Closed, ResolvedPublic2 Estimated Story Points

Assigned To
Authored By
Josve05a
Mar 3 2016, 1:49 AM
Referenced Files
F3906528: pasted_file
Apr 22 2016, 12:33 AM
F3906278: Simulator Screen Shot Apr 22, 2016, 12.51.40 AM.png
Apr 21 2016, 10:52 PM
F3870370: English-Hebrew.PNG
Apr 13 2016, 9:13 PM
F3870376: English.PNG
Apr 13 2016, 9:13 PM
F3870372: Hebrew-English.PNG
Apr 13 2016, 9:13 PM
F3851021: Add outline.pdf
Apr 8 2016, 10:12 PM
F3851022: Delete Reveal outline.pdf
Apr 8 2016, 10:12 PM
F3803549: remove.png
Mar 31 2016, 4:16 PM

Description

Summary:
It should be easy to remove languages in the on-boarding process. It can be achieved by having "remove" buttons upfront.

Spec:

  • The remove buttons will be on the left side of the list item for all list items except the first one. whatever the first one is.
  • User can re-order list items. even after reorder, the first item will not have a remove icon and will not be removable.
  • While reordering, the remove button will disappear and come back after release of drag
  • when tapped on remove icon, the list item will slide to left and reveal delete action, remove button will disappear here as well. to restore, user either can swipe back or tap anywhere else.

Mocks:

remove-languages.png (1×750 px, 78 KB)

remove.png (1×750 px, 78 KB)

Event Timeline

Also need to add deletion to the "language picker" so that they can also be removed downstream as well.

JMinor renamed this task from Allow removal of language in the FTUE-guide to Allow removal of language.Mar 24 2016, 8:21 PM
JMinor edited projects, added iOS-app-v5.0.2-Rubiks; removed iOS-app-v5.1.0-Saturn.
JMinor renamed this task from Allow removal of language to As a new user I want to easily remove unneeded languages.Mar 24 2016, 8:38 PM
JMinor raised the priority of this task from Medium to High.Mar 30 2016, 9:44 PM

@JMinor one open question

Is it possible to delete "English" (or whatever the device base language)?

if it is possible then we can still run into the problem of deleting all languages.

or is it based on the position in the stack? language at position 1 cannot be removed? others can be?

If there is no technical cost/risk then I'd say:

  • User always must have at minimum one language (aka "primary site language" can not be deleted)
  • On first start the current device language should be used as the primary site language
  • If the user then "demotes" the device language so it is not the primary site language, they can then delete it
  • If the user subsequently changes device language, the app chrome should use the new OS level language, but none of the site or search language settings should change

I leave to you (aka design) how to indicate that the "primary" language is not deletable or prevent the user from trying to delete it.

Nirzar renamed this task from As a new user I want to easily remove unneeded languages to As a new user It should be to remove languages while I'm on-boarding the app.Mar 31 2016, 4:16 PM
Nirzar updated the task description. (Show Details)
Nirzar removed a project: iOS-app-v5.0.2-Rubiks.
JMinor renamed this task from As a new user It should be to remove languages while I'm on-boarding the app to As a new user I should be to remove languages while I'm on-boarding the app.Apr 7 2016, 5:51 PM
JMinor set the point value for this task to 2.

hey @Fjalapeno, here are the assets for adding/removing languages during the onboarding


Josve05a renamed this task from As a new user I should be to remove languages while I'm on-boarding the app to As a new user I should be able to remove languages while I'm on-boarding the app.Apr 13 2016, 7:10 AM

Testing with iOS 9.3.1 the latest app build Wikipedia 5.0.3 (824). The remove button is listed on both items when there are multiple languages (English-Hebrew and Hebrew-English):

English-Hebrew.PNG (1×640 px, 60 KB)

Hebrew-English.PNG (1×640 px, 60 KB)

but when there is only one language (English) there is no remove icon:

English.PNG (1×640 px, 56 KB)
This is fixed.

@Nicholas.tsg just a note, I noticed the cards skipping design signoff. you can drag the cards to design signoff column after the QA.

@Mhurd there are some padding/margin issues. we can work it out offline.

Testing on iPhone with iOS 9.3.1 on the Wikipedia 5.0.3 (836) app.
This original issue is fixed. There is a remove icon for English now.
I filed a bug over another issue with it:
https://phabricator.wikimedia.org/T133361

@JMinor @Nirzar just wanted to triple check this is the expected behaviour – i.e., that user is able to remove the last remaining language?

My understanding was that once user only has 1 language remaining the icon to delete it disappears, per the way this works on iOS general language settings.

Simulator Screen Shot Apr 22, 2016, 12.51.40 AM.png (1×750 px, 61 KB)

Yes, here is what I thought we had, in spec form:

Given that I am on the language edit screen
When I have only one language
Then I should not see the delete control

There are also a few visual design tweaks (padding etc) as well listed below. I can go through in more detail with @Mhurd tomorrow/Mon too if that is faster.

  • each language cell should have a top and bottom border
  • language tableview should be centre-aligned with width of 243px
  • add and delete icons when shown appear to the RHS of the corresponding row (with a 10px gap between icon and row)
  • "Choose your preferred language to search Wikipedia" text looks like it is in Regular instead of the expected 'Light' font-weight

pasted_file (668×378 px, 45 KB)

@RHo's design tweaks have been consolidated on this ticket: T130875

Resolving this as functionally complete.