Page MenuHomePhabricator

[L] Year in Review - Build initial paging UI component
Closed, ResolvedPublic

Assigned To
None
Authored By
Tsevener
Sep 24 2024, 9:55 PM
Referenced Files
F57613318: CleanShot 2024-10-14 at 17.26.46.png
Oct 14 2024, 3:30 PM
F57613311: CleanShot 2024-10-14 at 17.23.49.png
Oct 14 2024, 3:30 PM
F57613301: CleanShot 2024-10-14 at 17.19.31.png
Oct 14 2024, 3:30 PM
F57613309: CleanShot 2024-10-14 at 17.21.32.png
Oct 14 2024, 3:30 PM
F57613293: CleanShot 2024-10-14 at 17.14.06.png
Oct 14 2024, 3:30 PM
F57613295: image.png
Oct 14 2024, 3:30 PM
F57583186: image.png
Oct 2 2024, 4:47 PM
F57562738: Screenshot 2024-09-27 at 11.55.56 AM
Sep 27 2024, 4:56 PM

Description

Background

The Apps team plans to explore if a personalized Wikipedia Year in Review feature that displays insights about a user's reading history, editing history, and donation history is engaging for App users, and inspires them to make a donation to the Wikimedia Foundation.

Task

This task is to build the UI paging component view for the Year in Review paging flow to match designs in Figma. A separate task for the entry point, introduction and closing screens, and the data populating this paging component will be created later. For now a new item in the profile view as an entry point will suffice (badge is not necessary at this point).

Screenshot 2024-09-27 at 11.55.56 AM (379×182 px, 27 KB)

Notes:

  1. New component UI work must live in WMFComponents. Component must theme properly and fonts must scale properly.
  2. The elements aligned to the bottom (Continue, Learn more buttons, Hashtag card, Share button, paging indicators, Next button) might need to pin to the bottom. The image, title text, and body text will likely need to be scrollable for smaller screens. (Confirm this assumption with design).
  3. Build with SwiftUI if possible.
  4. Do not add any animations yet, just a static image for the component.

Before we merge the PR: We need to either remove the entry point in profile, or add a developer feature flag for the entry point (task TBD).

    1. Test Notes
  • To have entry point, test in build 170 OR uncomment lines 178-199 in WMFProfileViewModel - this PR has the entry point commented out
QA Notes
  • Install app
  • Ensure device region in iOS settings is set to Mexico or Italy
  • Ensure primary app language is set to Spanish, Italian or English
  • Tap through rabbit hole to build up 6+ articles in the History tab if needed
  • Enable year in review feature flag in app developer settings menu
  • Background / foreground the app to trigger report creation.
  • Year in Review entry point should appear in Profile menu
  • Test year in review slides according to Figma. Done button, next button, theming, and dynamic type should work. Note the final donate slide (T376073) will not yet be in place.

Event Timeline

Tsevener updated the task description. (Show Details)
Tsevener renamed this task from Wikiwrapped - Build initial paging UI component to [L] Wikiwrapped - Build initial paging UI component.Oct 1 2024, 6:09 PM
GOlson-WMF added a subscriber: scblr.

@scblr - build 170 whenever you're ready! Note: I didn't copy the text directly from the Figma so it's dummy data for now but will be ripped out soonish

@GOlson-WMF a note that we would like the entry point to display for both logged-in and logged-out users

image.png (1×1 px, 177 KB)

@HNordeenWMF - will be added once we have the entry point as a "real" one - right now to keep users from accessing it, it'll be commented out so just using 1 for now :)

HNordeenWMF renamed this task from [L] Wikiwrapped - Build initial paging UI component to [L] Year in Review - Build initial paging UI component.Oct 4 2024, 3:14 PM
HNordeenWMF updated the task description. (Show Details)

Thanks @GOlson-WMF – great work so far!

In addition to @HNordeenWMF’s earlier point to show the entry point to all users, I noticed the following:

1) Please group the "Year in Review" menu item with the "Donate" option to streamline the menu architecture:

DesignvsImplementation
image.png (1×786 px, 65 KB)
CleanShot 2024-10-14 at 17.14.06.png (2×1 px, 184 KB)

2) Streamline horizontal alignment of labels, icons, and slider dots in the bottom section:

DesignvsImplementation
CleanShot 2024-10-14 at 17.21.32.png (1×786 px, 342 KB)
CleanShot 2024-10-14 at 17.19.31.png (2×1 px, 721 KB)

3) Highlight the active slide with the accent (blue) color:

CleanShot 2024-10-14 at 17.23.49.png (2×1 px, 218 KB)

4) Likely not part of this PR, but the info icon is missing from the view:

CleanShot 2024-10-14 at 17.26.46.png (1×786 px, 120 KB)

PS: I tried to study all the changes discussed last week, but please review my comments, @HNordeenWMF, in case I missed something.

@GOlson-WMF

In addition to @HNordeenWMF’s earlier point to show the entry point to all users, I noticed the following:

Please group the "Year in Review" menu item with the "Donate" option to streamline the menu architecture:

For these two pieces, I did this work in my PR for T376042, so no need to address those in this task.

  1. Likely not part of this PR, but the info icon is missing from the view:

We have a separate task for adding the info-icon/tooltip T376355. It'd be ideal if we don't add the visual element until we're able to have the tooltip/link working (as it's a nice-to-have).

Toni took care of 1, fixed 2 and 3, and 4 will be in that other ticket :)

Build #174

@GOlson-WMF and I tweaked this in real-time together – looks great now!

Tsevener added a subscriber: GOlson-WMF.
ABorbaWMF subscribed.

Testing on 7.6.2 (4337). This is working for me on iPhone for both personalized and collective slides. This is not working for me on iPad. On iPad, I only see the collective slide regardless of the number of articles in the history.