Page MenuHomePhabricator

Remove popup indicator on page load and associated configuration
Closed, ResolvedPublic2 Estimated Story PointsBUG REPORT

Description

Per T335307 this should be removed 2 weeks after usage.
Since T335307 was deployed on Monday 8th May, this should be removed on Monday 22nd May.
We can use a train deployment to make this happen. Doesn't matter if we have a few more days.

TODO

  • Remove the config $wgVectorLimitedWidthIndicator
  • Remove the indicators that show on page load
  • Retain indicators that show when the button is clicked

QA steps

  • Clicking the button should show a popup.
  • A popup should never shown on page load to anonymous users.

Sign off steps

Event Timeline

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

[mediawiki/skins/Vector@master] Remove limited width popup that shows on page load

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

LGoto set the point value for this task to 2.May 11 2023, 5:54 PM

Change 922160 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Remove limited toggle notification on page load and VectorLimitedWidthIndicator config

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

bwang removed bwang as the assignee of this task.May 23 2023, 4:42 PM
bwang subscribed.

Change 917398 abandoned by Jdlrobson:

[mediawiki/skins/Vector@master] Remove limited width popup that shows on page load

Reason:

See https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/922160

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

Change 922160 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Remove limited toggle notification on page load

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

Edtadros subscribed.

Test Result - Beta

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

Test Artifact(s):

QA Steps

✅ AC1: Clicking the button should show a popup.

1.gif (736×1 px, 769 KB)

✅ AC2: A popup should never shown on page load to anonymous users.

Screen Recording 2023-05-30 at 4.43.10 PM.mov.gif (748×1 px, 1 MB)

Change 926626 had a related patch set uploaded (by Kimberly Sarabia; author: Kimberly Sarabia):

[operations/mediawiki-config@master] Remove VectorLimitedWidthIndicator

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

KSarabia-WMF updated the task description. (Show Details)

Change 926626 merged by jenkins-bot:

[operations/mediawiki-config@master] Remove VectorLimitedWidthIndicator

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

Mentioned in SAL (#wikimedia-operations) [2023-06-08T20:05:23Z] <ladsgroup@deploy1002> Started scap: Backport for [[gerrit:926626|Remove VectorLimitedWidthIndicator (T336197)]]

Mentioned in SAL (#wikimedia-operations) [2023-06-08T20:06:49Z] <ladsgroup@deploy1002> ladsgroup and ksarabia: Backport for [[gerrit:926626|Remove VectorLimitedWidthIndicator (T336197)]] synced to the testservers: mwdebug2001.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug1001.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-06-08T20:12:55Z] <ladsgroup@deploy1002> Finished scap: Backport for [[gerrit:926626|Remove VectorLimitedWidthIndicator (T336197)]] (duration: 07m 32s)