Page MenuHomePhabricator

Limited-width toggle close button overlaps with text
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

Steps to replicate the issue

  • Click the limited-width toggle button

What happens?:

screenshot_2023-05-09_at_10.22.25_am.png (268×706 px, 29 KB)

What should have happened instead?:

The text should not overlap the close button

Other information

The close button only appears if the my_wikilimited-width-aware cookie is not set.

Event Timeline

Change 917899 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Add padding to limited-width toggle to account for close icon

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

Change 917899 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Add padding to limited-width toggle to account for close icon

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

ovasileva set the point value for this task to 2.May 9 2023, 4:35 PM

Change 917733 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@wmf/1.41.0-wmf.7] Add padding to limited-width toggle to account for close icon

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

Change 917734 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@wmf/1.41.0-wmf.8] Add padding to limited-width toggle to account for close icon

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

Change 917733 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.41.0-wmf.7] Add padding to limited-width toggle to account for close icon

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

Change 917734 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.41.0-wmf.8] Add padding to limited-width toggle to account for close icon

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

Mentioned in SAL (#wikimedia-operations) [2023-05-09T20:22:52Z] <urbanecm@deploy1002> Started scap: Backport for [[gerrit:917733|Add padding to limited-width toggle to account for close icon (T336274)]], [[gerrit:917734|Add padding to limited-width toggle to account for close icon (T336274)]]

Mentioned in SAL (#wikimedia-operations) [2023-05-09T20:24:24Z] <urbanecm@deploy1002> urbanecm and jdrewniak: Backport for [[gerrit:917733|Add padding to limited-width toggle to account for close icon (T336274)]], [[gerrit:917734|Add padding to limited-width toggle to account for close icon (T336274)]] synced to the testservers: mwdebug2001.codfw.wmnet, mwdebug1001.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-05-09T20:31:51Z] <urbanecm@deploy1002> Finished scap: Backport for [[gerrit:917733|Add padding to limited-width toggle to account for close icon (T336274)]], [[gerrit:917734|Add padding to limited-width toggle to account for close icon (T336274)]] (duration: 08m 59s)

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Ventura
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

✅ AC1: Limited-width toggle close button should not overlap with text

Screenshot 2023-05-16 at 2.46.54 PM.png (656×1 px, 199 KB)

Jdlrobson claimed this task.

This can skip QA in production since we'll be disabling it next week.