Page MenuHomePhabricator

[M] Navigate away from new profile view
Closed, ResolvedPublic3 Estimated Story Points

Description

Background

As part of the Navigation refresh, T373712, we are creating a profile view that will be accessible from both article view and the main view for quick access to notifications, watchlist, and other items throughout the entire app.

Requirements
  • When tapping an item in profile view, dismiss, then push on or present destination view controller.
  • Clicking on notifications, user page (web view), talk page, watchlist, logout, settings, and donate for logged-in users takes you to those locations (not nested within the profile view)
  • If a user chooses to login or logout, they are returned to the page they were on before (profile view does not need to be open)
  • Functions for landscape and iPad
Designs

n/a

Event Timeline

Seddon renamed this task from Navigate away from profile view to [M] Navigate away from new profile view.Sep 6 2024, 12:58 PM
Seddon set the point value for this task to 3.

@Tsevener

Great work and no surprises here; it is implemented as discussed regarding back behavior and labeling. We can think/talk about removing the "Back" label app-wide in the future.

@Mazevedo I am running seeing 2 issues while clicking around

Issue 1
Steps to recreate:

  • Open article
  • Open profile and navigate to watchlist
  • Navigate back to article
  • Click search (from top right corner), then "cancel"

What happens

  • Profile & Search disappear from top

Issue 2
Steps to recreate:

  • Open Explore
  • Open profile and navigate to watchlist
  • Navigate back to Explore
  • Open article from explore
  • Navigate back to Explore

What happens

  • "Wikipedia" disappears from top

Edit: iOS 18.1 Beta, iPhone, 7.6.0 (4183)

I'm kicking off an Experimental build @HNordeenWMF. I don't think I fixed it entirely, but maybe it will be at least improved. I'm sorry that I'm such a broken record here, but this is our custom navigation bar acting up again. We need to prioritize T363606 as soon as we get a moment, or else these bugs will continue to chew up time as we work on features.

Experimental build will be 7.6.0 (171).

@Tsevener this looks good to me, the Profile/Search disappearing is fixed. The Logo disappearing is still happening but that is less concerning as it's not a button.

This fix has been merged - can be tested in TestFlight 7.6.0 (4193).