Page MenuHomePhabricator

Increased font-size (>=20px) causes TOC to overlap text
Closed, ResolvedPublic3 Estimated Story Points

Description

Description

In Google Chrome if you go to Settings/Appearance (you can get there by putting this in your URL bar chrome://settings/appearance), and increase the font-size to either "Large" or "Very large", then view a Wikipedia article with a table of contents (or with the main menu open), you will see the following:

LargeVery large
Screen Shot 2022-07-26 at 10.54.53 AM.png (2×3 px, 3 MB)
Screen Shot 2022-07-26 at 10.55.02 AM.png (2×3 px, 3 MB)
Screen Shot 2022-07-26 at 10.55.27 AM.png (2×3 px, 3 MB)
Screen Shot 2022-07-26 at 10.55.45 AM.png (2×3 px, 3 MB)

QA steps

  • Test the reported steps in Chrome
  • In Firefox change the font-size to 20px and check the sidebar doesn't overlap

Notes

This issue does not occur if you use the "Zoom In" feature within the "View" menu:

Screen Shot 2022-07-26 at 10.57.46 AM.png (738×816 px, 355 KB)

Screen Shot 2022-07-26 at 10.58.02 AM.png (2×3 px, 3 MB)

Font size can be changed in Firefox like so:

Screen Shot 2022-07-26 at 8.14.07 PM.png (222×1 px, 29 KB)

QA Results - Prod

ACStatusDetails
1T313817#8230983
2T313817#8230983

Event Timeline

ovasileva triaged this task as Medium priority.Jul 26 2022, 3:00 PM
alexhollender_WMF raised the priority of this task from Medium to Needs Triage.Jul 26 2022, 3:00 PM
alexhollender_WMF updated the task description. (Show Details)
Aklapper triaged this task as Medium priority.Jul 26 2022, 3:38 PM
Jdlrobson renamed this task from [Chrome bug] Increased font-size causes TOC to overlap text to Increased font-size causes TOC to overlap text.Jul 26 2022, 7:14 PM
Jdlrobson renamed this task from Increased font-size causes TOC to overlap text to Increased font-size (>=20px) causes TOC to overlap text.
Jdlrobson claimed this task.
Jdlrobson updated the task description. (Show Details)
Jdlrobson raised the priority of this task from Medium to High.Jul 27 2022, 4:28 PM

This can be fixed by switching the grid definition from pixels to ems or rems. REM support is good (pretty much a a guarantee when using CSS grid) https://caniuse.com/rem

Screen Shot 2022-07-27 at 5.30.27 PM.png (1×2 px, 796 KB)

@ovasileva this seems pretty bad to me so suggest treating fixing this as a high priority.

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

[mediawiki/skins/Vector@master] Express sidebar width in pixel for font size

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

Change 821176 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Express sidebar width in pixel for font size

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

QA findings

Testing on https://en.wikipedia.org/wiki/Edward_Ruscha
Macbook Pro

Chrome: Very Large

Screen Shot 2022-08-18 at 5.22.22 PM.png (1×3 px, 1 MB)

Firefox: Font size 20

Screen Shot 2022-08-18 at 5.54.51 PM.png (1×2 px, 741 KB)

Sidebar looks good in Chrome
Sidebar is overlapping in Firefox.

Can you look at this again, I think the train was reverted so this shouldn't be a problem now.

Edtadros subscribed.

Test Result - Prod

Status: ✅ PASS
Environment: dewiki
OS: macOS Monterey
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

In Google Chrome if you go to Settings/Appearance (you can get there by putting this in your URL bar chrome://settings/appearance), and increase the font-size to either "Large" or "Very large", then view a Wikipedia article with a table of contents (or with the main menu open), you will see the following:

Test the reported steps in Chrome
In Firefox change the font-size to 20px and check the sidebar doesn't overlap

✅ AC1: In Chrome, the sidebar should not overlap article content

Screen Shot 2022-09-12 at 6.52.56 PM.png (630×1 px, 172 KB)

Screen Shot 2022-09-12 at 6.52.47 PM.png (664×1 px, 190 KB)

Screen Shot 2022-09-12 at 6.48.58 PM.png (730×1 px, 195 KB)

Screen Shot 2022-09-12 at 6.48.42 PM.png (680×1 px, 189 KB)

✅ AC2: In Firefox, the sidebar should not overlap article content

Screen Shot 2022-09-12 at 6.54.21 PM.png (842×1 px, 278 KB)

Screen Shot 2022-09-12 at 6.54.12 PM.png (1×1 px, 435 KB)

Looks good, resolving