Page MenuHomePhabricator

Standardized header: Places
Closed, ResolvedPublic1 Estimated Story Points

Assigned To
None
Authored By
HNordeenWMF
Jan 15 2025, 9:10 PM
Referenced Files
F58385455: image.png
Feb 11 2025, 10:44 AM
F58385434: IMG_1028.PNG
Feb 11 2025, 10:44 AM
F58385433: IMG_1029.PNG
Feb 11 2025, 10:44 AM
F58385402: 0847C16A-0DB7-4E91-A776-35B146AA0433_1_102_o.jpeg
Feb 11 2025, 10:44 AM
F58385404: C3916FD2-C828-4527-A5E6-EFE1F000310A_1_102_o.jpeg
Feb 11 2025, 10:44 AM
F58384292: Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.33.png
Feb 10 2025, 10:43 PM
F58384290: Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.41.png
Feb 10 2025, 10:43 PM
F58384181: IMG_1007.PNG
Feb 10 2025, 9:18 PM

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.

Requirements
  • Add access to profile onto Places view (ignore Tabs icon for now)
  • Update header design to match Figma
  • The header should remain sticky when user scrolls within map, or list view (different from other main views)
  • Must function on both iPhone and iPad
  • Must function in Landscape mode
Designs (for reference, check Figma for most up-to-date)
iPhoneiPad
image.png (1×786 px, 586 KB)
image.png (2×1 px, 2 MB)

Event Timeline

Tsevener renamed this task from Standardized header: Places to [S] Standardized header: Places.Jan 16 2025, 7:08 PM
Seddon renamed this task from [S] Standardized header: Places to Standardized header: Places.Jan 17 2025, 1:10 AM
Seddon set the point value for this task to 1.

Just noticed a minor difference in the search bar regarding icon spacing and label consistency @Mazevedo (compare to the one in Explore, Saved or Search):

PlacesvsSearch (for example)
IMG_1008.PNG (2×1 px, 1 MB)
IMG_1007.PNG (2×1 px, 193 KB)
  • Spacing between icon + label is too large (see the other search bars as reference)
  • Consistent sentence case label: Search places as opposed to Search Places

Thanks!

@scblr @HNordeenWMF - the spacing issue seems to be the component adjusting the text to the space available - in the screenshots below, in Portuguese, the text is longer on both screens, and the spacing is consistent. We have no control over that part of the component in this case.

Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.41.png (2×1 px, 155 KB)

Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.33.png (2×1 px, 1 MB)

@scblr @HNordeenWMF - the spacing issue seems to be the component adjusting the text to the space available - in the screenshots below, in Portuguese, the text is longer on both screens, and the spacing is consistent. We have no control over that part of the component in this case.

Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.41.png (2×1 px, 155 KB)

Simulator Screenshot - iPhone 16 - 2025-02-10 at 19.39.33.png (2×1 px, 1 MB)

Hmm, this is weird! It doesn’t adjust the space in current production! ⬇️

C3916FD2-C828-4527-A5E6-EFE1F000310A_1_102_o.jpeg (2×1 px, 226 KB)
0847C16A-0DB7-4E91-A776-35B146AA0433_1_102_o.jpeg (2×1 px, 619 KB)

I checked French and German in Experimental build 7.7.1.221 , where the problem (gap between icon + label) is occurring as well with various string lengths:

IMG_1029.PNG (2×1 px, 1 MB)
IMG_1028.PNG (2×1 px, 1 MB)

In Portuguese (Brazil) as well on my device! ⬇️ 🤔

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

Already have a task for that fix! T383464

Thx Haley – oversaw that 🙈

HNordeenWMF removed Mazevedo as the assignee of this task.
HNordeenWMF added a subscriber: Mazevedo.