Page MenuHomePhabricator

Create bottom module that shows Did You Know or Because You Read content
Closed, ResolvedPublic3 Estimated Story Points

Description

Background

As part of “More dynamic tabs,” we want a lightweight recommendations module pinned to the bottom of the Tabs Overview screen. The module helps readers discover content when they have little recent activity and adapts based on activity level.

Requirements

Entry point / placement

  • The module is pinned to the bottom of the Tabs Overview screen .
  • Module shows only for users in test groups B or C .

State logic (tab count)

  • Define tab count as number of open article tabs excluding the Main Page tab.
  • If tab count < 2 → show “Did You Know” (DYK) module.
  • If tab count ≥ 2 → show “Because you read …” module.

Because You Read source

  • The most recent 2 tabs should be the seeds for the recommendation
  • Tapping a recommendation opens in a new tab and takes the user to the article.
  • There should be 3-5 articles in a carousel view

Hide / overflow
Note: Will be done in T404796

  • When selected, the recommendations module disappears immediately and stays hidden for that user until they re-enable via overflow menu

Behavioral details

  • [Nice to have] Long-pressing on any recommendation or DYK link should show standard Peek/Preview (same behavior as Explore feed).
  • Live-update back to DYK if someone closes out all tabs / all except 1 non-mainpage tab
  • Do not live-update with new recommendations if someone close out articles that were the seeds of their recommendations, but still has 2+ non-main page tabs tabs open
  • Clicking on any recommended article should open that article in a new tab
  • Respect current theme (light/dark/sepia/black) and dynamic type.

Copy

See Designs

Designs

https://www.figma.com/design/5gHmEo8IIFxlPfYM6bppzz/iOS---%3E-More-Dynamic-Tabs?node-id=1418-5402&t=TvVPKxZHLdbEZB0O-1

  1. Engineering Notes

Event Timeline

Mazevedo updated the task description. (Show Details)
Mazevedo set Final Story Points to 8.

Please design review on Staging 7.8.4 (109).

Update:

Please design review on Staging 7.8.4 (110) for Group B.
Please design review on Staging 7.8.4 (111) for Group C.

Before your first app launch, ensure your device is in a target experiment region (I use Germany).

Group B: Staging 115
Group C: Staging 116

SChekfa-WMF subscribed.

@GOlson-WMF

  1. i'm noticing that for DYKs that are longer than three lines, the text is cut off and it's impossible to read the entire fragment. is it possible to limit the DYKs to sentences that will fit in those three lines? or would it be easier to link the user perhaps to the actual DYK to read the entire fragment?
    IMG_8581.PNG (2×1 px, 373 KB)
  1. i'm noticing that some DYKs, even though they're less than 3 lines long, are getting truncated. can you please look into why this is happening?
    View recent photos.png (2×1 px, 163 KB)
  1. for the DYKs, there seems to be excess padding above the title when the text encompasses less than 3 lines. can we ensure this is at 16px regardless of how long the text is? see above for how it looks when there's one line, and here's how it looks at 2:
    IMG_8586.PNG (2×1 px, 413 KB)
  1. the type token referenced by the DYK title seems to be off — can you please ensure it is the same as BYR, Subhead/SF Pro/Medium?
  1. the drop shadow for BYR feels a bit heavy compared to the design. can you please ensure the shadow is implemented per the spec?
  1. speaking of drop shadow, there is a cut in the shadow for BYR:
    image.png (658×952 px, 85 KB)
    can you please ensure this is removed? TY!

1 - we don't really have any control over this content, nor do we really get a link for it unfortunately. We can increase the max line size, but it would have to end up getting taller. What route would you like to go with that @SChekfa-WMF

Looking into the rest! Number 6 may be difficult due to it being the safe area of the phone, but hopefully softening it will fix that

Hi @TLessa-WMF, could you review these items Sarah commented on?
We have a new build with these updates 7.8.4 (5850)

@TLessa-WMF and @Mazevedo - did y'all get a chance to chat on this?

@TLessa-WMF and @Mazevedo - did y'all get a chance to chat on this?

@GOlson-WMF we just did, I'll update that with the current work I'm doing for T407669

Appears to be fixed on 7.8.4 (5853)

Tested on iPhone 16 on iOS 26, iPad 12.9 on iPadOS 26, and iPhone 11 on iOS 18. Tested on all themes