Page MenuHomePhabricator

[L] Create explore feed card for Image Recs
Open, LowPublic

Assigned To
None
Authored By
Mazevedo
Feb 6 2024, 10:07 PM
Referenced Files
F42752130: Screenshot 2024-03-19 at 8.59.57 AM
Mar 19 2024, 2:00 PM
F42752052: Screenshot 2024-03-19 at 8.56.04 AM
Mar 19 2024, 1:57 PM
F42752050: Screenshot 2024-03-19 at 8.55.20 AM
Mar 19 2024, 1:57 PM
Restricted File
Mar 5 2024, 6:13 PM
Restricted File
Mar 5 2024, 6:13 PM
Restricted File
Mar 5 2024, 6:07 PM
Restricted File
Mar 5 2024, 6:07 PM
Restricted File
Mar 5 2024, 1:33 PM

Description

Create the explore feed card according to the Figma reference

Screenshot 2024-02-06 at 19.03.00.png (380×768 px, 62 KB)

Requirements

  • Card should never be the first on the explore feed
  • User needs to have at least 50 edits to see the card
  • Check if the wiki has the suggested edits task before displaying the card
  • Add the card only once for the Main app language

Related Spike: T355328

Items:

  • Create card UI
  • Create displaying logic based on requirements above
Testing

Please test in TestFlight Wikipedia app 7.5.0 (3509).

@ABorbaWMF Note that once we saw the card not appear due to the Explore feed settings toggle being off by default upon first launch. We can no longer repro though. Please keep an eye out for this and let us know if you can get a screen recording of this issue.

Event Timeline

Tsevener renamed this task from [Task] Create explore feed card for Image Recs to [L] Create explore feed card for Image Recs.Feb 28 2024, 4:34 PM

Hi @OTichonova. Since your Suggested Edits explore card mock has the 3-dot-icon for hiding, I'm adding in the ability to hide the card via Explore feed preferences (this seems to match Android as well).

Can you give me an icon, title text, subtitle text, and icon foreground and background colors for this row?

Screenshot 2024-02-28 at 3.53.51 PM (939×497 px, 262 KB)

For users with multiple language wikis in their app, how does it generally work with displaying Suggested Edits cards? For example if they only had rights on German Wikipedia, but had English enabled as well: Would we add a "From German Wikipedia" subtitle below the title of Suggested edits card in the explore feed?

Screenshot 2024-03-01 at 11.20.42.jpeg (327×1 px, 131 KB)

Can you give me an icon, title text, subtitle text, and icon foreground and background colors for this row?

Hi @Tsevener
I realized that the page looks slightly different if one has a single 'Wikipedia language' or multiple 'Wikipedia languages' in the app.

Single languageMultiple languages
{F42400611}{F42400613}
{F42400516}{F42400519}

Icon: OOUI pencil

Icon color: blue600
Background color: blue100

Title: Suggested edits
Subtitle text for single language: Suggestions to add content to Wikipedia
Subtitle text for multiple languages: Not language specific

For users with multiple language wikis in their app, how does it generally work with displaying Suggested Edits cards? For example if they only had rights on German Wikipedia, but had English enabled as well: Would we add a "From German Wikipedia" subtitle below the title of Suggested edits card in the explore feed?

Screenshot 2024-03-01 at 11.20.42.jpeg (327×1 px, 131 KB)

I asked Android about this. They display a single Suggested Edits card per "day" of the feed, and it's for the primary language of the app. So it's sort of language-specific in that it's only suggesting things against the primary app language, but it's not language-specific in that you do not see additional suggested edits cards per language in the feed. I don't think Android shows any language indicator in their Explore card.

Tsevener updated the task description. (Show Details)
Tsevener subscribed.

Hi @Tsevener, it is looking good. 1 small tweak. I realized now that the header size in the other cards is 16pts (Callout) not 15pts (Subhead). Could the header text inside the card -> 'Add an image' be increased to 16pts (Callout)? Sorry about that.

Additionally, when I set Arabic as my primary language or Hebrew the card doesn't show up in the experimental build 88, should I be testing RTL in a different experimental build?

cc @Mazevedo

@OTichonova My guess is that you don't have 50+ edits on those wikis. Here are some screenshots from temporarily removing that requirement:

LTR device language, RTL app primary language:

Screenshot 2024-03-19 at 8.55.20 AM (1×559 px, 309 KB)

RTL device language, RTL app primary language:

Screenshot 2024-03-19 at 8.56.04 AM (1×559 px, 266 KB)

Translations aren't in yet.

@OTichonova

Screenshot after bumping font size up:

Screenshot 2024-03-19 at 8.59.57 AM (1×559 px, 291 KB)

Hi @Tsevener

  • Ah okay thanks for the RTL
  • Typesize looks good!