Page MenuHomePhabricator

iPad floating tab bar search icon
Open, LowPublic5 Estimated Story Points

Description

Background

As part of the ongoing Navigation refresh, we are standardizing the design of the headers throughout the app on iPhone and iPad, to standardize and allow room for upcoming features. There are several improvements specific to the iPad floating toolbar, which displays on all views of the app currently.

Requirements
  • Replace "Search" title in toolbar with magnifying glass icon from SF Symbols
  • Match design to figma
Designs

Figma

image.png (2×1 px, 1 MB)

Engineering Notes

TS - See Apple documentation on how to do this. We need to set AppViewController's tabs property, and use a UISearchTab for the final tab. Re-check all programatic navigation after doing this. The last time I tried this, search results navigation broke (see https://phabricator.wikimedia.org/T363606#10408281).

Event Timeline

Tsevener renamed this task from iPad floating tab bar search icon to [L] iPad floating tab bar search icon.Jan 16 2025, 7:13 PM
Mazevedo renamed this task from [L] iPad floating tab bar search icon to iPad floating tab bar search icon.Jan 23 2025, 6:31 PM
Mazevedo set the point value for this task to 5.

@HNordeenWMF Is the change specific to iPad? I'm assuming yes but wanted to confirm.

I made a change and it seems to be working on iPad but its broken on iPhone.

Also I've fixed the issue mentioned in the engineering notes section in description of the task.

Here's the demo

Hi @Chandrapg - correct, this should only be changed on iPad.
I'm unable to see the demo :/ could you try attaching the file again? Sometimes you need to check the top-right-section "Referenced Files" of the task and "Attach file" if it's showing in red.

@Chandrapg Nice! This is looking good to me so far. Can you put up a PR so we can dig into it further? Thank you!

PR: https://github.com/wikimedia/wikipedia-ios/pull/5202

I've tested regular scenarios like searching in every tab, switching tabs on iPad and also verified on iPhone that we don't use these changes. But I'm sure I might have missed other areas.

@scblr This will be in Experimental build 7.7.2 (224) for design review.

Looks and works well from a design perspective — thanks @Chandrapg and @Tsevener 👏