Page MenuHomePhabricator

Update navigation bar
Closed, ResolvedPublic1 Estimated Story Points

Description

  • Reference https://github.com/wikimedia/wikipedia-ios/pull/5176 for how to pull an accurate auth state, as our approach changed a lot in PR review for that.
  • If auth state indicates ip or temporary, add new button with expected icon to navigation bar in editor. Present sheet from T374010 upon tap
  • (If needed) Update edit notices icon to orange.

Test Steps

Using a temporary account, Test language
Navigate to article
Press pencil button (unlocked)
Make sure temporary account sheet shows, press Done
Ensure the temporary account icon shows, tap it
Ensure temporary account sheet shows
Repeat steps with IP account

Experimental Build 238

Event Timeline

Seddon triaged this task as Low priority.Sep 6 2024, 2:07 PM
Seddon moved this task from Needs Triage to Up next on the Wikipedia-iOS-App-Backlog board.
Tsevener updated the task description. (Show Details)
HNordeenWMF raised the priority of this task from Low to Medium.Dec 19 2024, 10:22 PM
Seddon renamed this task from [S] Update navigation bar to Update navigation bar.Jan 27 2025, 1:30 PM
Seddon set the point value for this task to 1.

@scblr - the navigation bar in total is different but outside the scope of this particular ticket, but here are videos of the sheets and notices!

Hey @GOlson-WMF – this mostly looks good – the secondary CTA’s could be moved slightly up, though (~ 8px). See the comparison below.

Design (first screen) vs Implementation (second and third screen):

secondary-ctas.png (1×2 px, 584 KB)

After this minor tweak, there’s no need to design review again – thanks!

This is ready to QA. Moving to Blocked / Waiting since it's hanging out in a feature branch (temp-accounts) until all temp accounts work is done.