Page MenuHomePhabricator

[Spike] Wikipedia logo and navigation link elements should scale accordingly with font-size increases
Closed, ResolvedPublic3 Estimated Story PointsSpike

Assigned To
Authored By
• alexhollender_WMF
Oct 15 2018, 2:29 PM
Referenced Files
F34646054: IMG_2953.jpg
Sep 17 2021, 1:46 PM
F34646016: IMG_2936.jpg
Sep 17 2021, 1:46 PM
F34646035: IMG_2942.jpg
Sep 17 2021, 1:46 PM
F34646015: IMG_2935.jpg
Sep 17 2021, 1:46 PM
F34646106: IMG_2960.jpg
Sep 17 2021, 1:46 PM
F34646103: IMG_2962.jpg
Sep 17 2021, 1:46 PM
F34646058: IMG_2948.jpg
Sep 17 2021, 1:46 PM
F34646109: IMG_2956.jpg
Sep 17 2021, 1:46 PM

Description

This was discovered here: T204807#4652971

Description

iOS has a setting that allows you to increase the font-size on your phone. The Wikipedia mobile site is responsive to that setting (i.e. the font gets bigger), however there are some non-text elements that don't scale accordingly.

Wikipedia logonav icons
IMG_2012.png (750×1,334 px, 161 KB)
Screen_Shot_2018-10-09_at_1.06.16_PM.png (790×1,752 px, 159 KB)

These two elements were noticed because they are quite prominent, however it might be worth going through the website more extensively looking for other elements that don't scale.

Environments

  • iOS / Safari & Chrome browsers

Questions to answer

  • Is this bug still reproducable
  • Is there anything we can do here?
  • If so, how much work is involved?

QA Results - Prod

ACStatusDetails
1✅T207038#7361971

Event Timeline

@Jdrewniak do you have any idea why these icons are not scaling? I don't know enough about how this iOS font handler works.

Side note, when I visit the settings page on iOS, changing font sizes seems to have no impact. Is this just me

@Jdrewniak do you have any idea why these icons are not scaling? I don't know enough about how this iOS font handler works.

Side note, when I visit the settings page on iOS, changing font sizes seems to have no impact. Is this just me

@Jdlrobson are you testing with iOS 9+? The dynamic type feature we enabled in T207789 will only take effect in iOS 9+ browsers...

Jdlrobson lowered the priority of this task from Medium to Low.EditedOct 30 2018, 10:00 PM
Jdlrobson added a subscriber: ovasileva.

@ovasileva also let me know if the priority here needs bumping. A spike would be the obvious next step for the font issue problems.

Jdlrobson renamed this task from [Bug] All UI elements should scale accordingly with font-size increases to [Bug] Mobile UI elements should scale accordingly with font-size increases.Aug 2 2021, 7:56 PM
Jdlrobson renamed this task from [Bug] Mobile UI elements should scale accordingly with font-size increases to [Bug] Wikipedia logo and navigation link elements should scale accordingly with font-size increases.
Jdlrobson renamed this task from [Bug] Wikipedia logo and navigation link elements should scale accordingly with font-size increases to [Spike] Wikipedia logo and navigation link elements should scale accordingly with font-size increases.Aug 23 2021, 5:02 PM
Jdlrobson added a project: Spike.
Jdlrobson updated the task description. (Show Details)
Restricted Application changed the subtype of this task from "Task" to "Spike". · View Herald TranscriptAug 23 2021, 5:02 PM

The solution should be technically quite simple and doesn't only affect iOS.
The logo is currently carrying width and height attributes, whose values are limited to pixel:
<img src="/static/images/mobile/copyright/wikipedia-wordmark-en.svg" width="119" height="18"">

Try by setting your font-size to anything different than default ('Medium') in browser settings, here in Google Chrome:

image.png (1,422×314 px, 26 KB)

Overriding those either with inline styles or a CSS selector to carry relative unit like rem should result in resized logos on all platforms according to user preferences.

I think the logo is the only thing that needs fixing here. See T207789 for how we could do that

Nav elements scale well. Both icons and text are using relative units.

Screen Shot 2021-09-03 at 9.19.06 AM.png (2,366×700 px, 141 KB)

Change 719599 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Logo scales with font-size

https://gerrit.wikimedia.org/r/719599

Change 719501 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Logos defined with pixels should be scaleable in browsers\"\"

https://gerrit.wikimedia.org/r/719501

Jdlrobson added a subscriber: Edtadros.

Change 719501 merged by jenkins-bot:

[mediawiki/core@master] Logos defined with pixels should be scaleable in browsers

https://gerrit.wikimedia.org/r/719501

@Jdlrobson I'm not sure what the acceptance criteria are, so here are my findings:

  1. Smallest

IMG_2918.PNG (1,284×2,778 px, 1 MB)

  1. Largest

IMG_2919.PNG (1,284×2,778 px, 954 KB)

  1. Largest using Accessibility

IMG_2920.PNG (1,284×2,778 px, 981 KB)

Moving to QA in Prod. Per standup, I will be testing on Italian wiki and looking at just the logo.

Test Result - Prod

Status: ✅ PASS
Environment: enwiki, itwiki
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

✅ AC1: Verify the wikipedia logo is resized as text is resized by iOS.
The logo size does increase/decrease as expected. I verified this in itwiki as well. The rest of the images below are for reference. As the text gets really large there are elements on the screen that overlap, and search requires scrolling to the right to see the magnifying glass icon. A user may not know it is there. Search text size doesn't increase.

Default Size
IMG_2939.jpg (1,284×2,650 px, 128 KB)
IMG_2935.jpg (1,284×2,655 px, 201 KB)
IMG_2936.jpg (1,284×2,646 px, 239 KB)
IMG_2937.jpg (1,284×2,652 px, 232 KB)
IMG_2933.jpg (1,284×2,650 px, 324 KB)
IMG_2934.jpg (1,284×2,653 px, 196 KB)
IMG_2938.jpg (1,284×2,650 px, 293 KB)
Smallest Size
IMG_2940.jpg (1,284×2,635 px, 120 KB)
IMG_2946.jpg (1,284×2,660 px, 284 KB)
IMG_2944.jpg (1,284×2,657 px, 215 KB)
IMG_2943.jpg (1,284×2,650 px, 200 KB)
IMG_2942.jpg (1,284×2,644 px, 186 KB)
IMG_2941.jpg (1,284×2,655 px, 292 KB)
IMG_2945.jpg (1,284×2,653 px, 213 KB)
Large Size
IMG_2947.jpg (1,284×2,643 px, 176 KB)
IMG_2950.jpg (1,284×2,650 px, 201 KB)
IMG_2949.jpg (1,284×2,657 px, 209 KB)
IMG_2953.jpg (1,284×2,655 px, 280 KB)
IMG_2954.jpg (1,284×2,658 px, 305 KB)
IMG_2952.jpg (1,284×2,650 px, 298 KB)
IMG_2951.jpg (1,284×2,644 px, 332 KB)
IMG_2948.jpg (1,284×2,655 px, 314 KB)
Largest Size
IMG_2955.jpg (1,284×2,661 px, 295 KB)
IMG_2962.jpg (1,284×2,678 px, 249 KB)
IMG_2958.jpg (1,284×2,643 px, 235 KB)
IMG_2957.jpg (1,284×2,641 px, 251 KB)
IMG_2960.jpg (1,284×2,658 px, 296 KB)
IMG_2959.jpg (1,284×2,657 px, 292 KB)
IMG_2961.jpg (1,284×2,672 px, 348 KB)
IMG_2956.jpg (1,284×2,664 px, 237 KB)

@Jdlrobson, I'm assigning this to you just to highlight the behavior at the largest text size. The issues with search text size, the search clear x button misalignment, and overlapping text are outside of the scope of just the logo, but I thought you would like to note them in case you feel they warrant a fix. If everything looks good let me know and I'll move it over to Ready for Signoff.

The logo looks great in all of these.
Thanks for the detailed analysis of all the other font sizes. Things don't look great there, but someone viewing the site could always toggle "view desktop mode" to clean that up so at least some workround exists.

Have created T291503 to capture potential future work.

Change 807204 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/core@master] resourceloader: Resolve SkinModule wordmark/tagline tech debt

https://gerrit.wikimedia.org/r/807204

Change 807204 merged by jenkins-bot:

[mediawiki/core@master] resourceloader: Resolve SkinModule wordmark/tagline tech debt

https://gerrit.wikimedia.org/r/807204