Page MenuHomePhabricator

Move History into Search
Closed, InvalidPublic5 Estimated Story Points

Assigned To
Authored By
HNordeenWMF
Jan 22 2025, 5:25 PM
Referenced Files
F58260153: image.png
Jan 23 2025, 6:32 PM
F58260144: image.png
Jan 23 2025, 6:31 PM
F58250190: image.png
Jan 22 2025, 5:28 PM
F58250193: image.png
Jan 22 2025, 5:28 PM

Description

NOTE: This should not be deployed to production until 15 days after 7.7.2 has released to 100% (so we have baselines for Article Source data to track the affect of this move)
Background

As part of the ongoing Navigation refresh, we plan to move the History functionality into the Search tab, and remove the "history" view from our main views (tabs).

Requirements
  • Show history below search within the Search Tab
  • Remove History from bottom tab bar
  • History's empty state should show within Search tab if user has no history
  • User must be able to clear History
  • Adjust spacing of remaining elements to match Figma Do not update icons to use SF icons yet
  • Must function on both iPhone and iPad
  • Must function in Landscape mode
  • At least the last 100 articles must be brought over for the user to see
  • Entire history should be retained/cleared as before for future Year in Reviews
  • Articles opened from "History" section of Search tab should be logged with an article source of History, not Search

Nice-to-have:

  • Upgrade History to Swift UI
Designs

Figma

image.png (812×374 px, 92 KB)
image.png (2×3 px, 1 MB)
Engineering Notes

Update: We have made initial progress on this with SwiftUI, please build off of branch feature/history-to-search. Old notes below.

We will be updating SearchViewController.swift, which is displayed in the search tab. Note that this same view controller is re-used in these places:

  • Explore > search bar search,
  • Editor > Insert Link search
  • Editor > Edit Link search
  • Article > Magnifying glass button
  • Article > search bar search (in A/B test)
  • In-app web view. Magnifying glass button (Profile > User Page for example)

Extra care must be taken to ensure regressions don't happen in those areas (History is embedded in search ONLY when in search tab).

Would be ideal if we can rebuild the history view in SwiftUI, then embed it's hosting controller within SearchViewController. If this proves too difficult, we can try embedded the existing HistoryViewController in SearchViewController.

Event Timeline

HNordeenWMF raised the priority of this task from Low to Medium.Feb 3 2025, 10:29 PM
Mazevedo added a subscriber: Tsevener.