Page MenuHomePhabricator

Enable pinning for anonymous users when page tools is enabled
Closed, ResolvedPublic2 Estimated Story PointsFeature

Description

Feature summary (what you would like to be able to do and where):

  • Enable pinning for anonymous users when page tools is enabled. Currently we hide the controls. We should show them.
  • There is no need to make the functionality persistent.

Benefits (why should this be implemented?):

Some power users would benefit from having a sticky page tools on the right hand side of the screen and easier access (scroll rather than click) to access the main menu.

QA steps

  1. Visit https://en.wikipedia.beta.wmflabs.org/wiki/Dog?vectorpagetools=1
  2. Open page tools menu in the page toolbar.
  3. Verify that the menu has the "move to sidebar" option and that it works.
  4. Open the main menu.
  5. Verify that the main menu has the "move to sidebar" option and that it works.

QA Results - Beta

ACStatusDetails
1T331657#8700648
2T331657#8700648

QA Results - Prod

ACStatusDetails
1T331657#8726893
2T331657#8726893

Event Timeline

ovasileva moved this task from Incoming to Current Quarter on the Web-Team-Backlog board.
ovasileva set the point value for this task to 2.Mar 9 2023, 6:36 PM

Change 897998 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@master] Enable the pinning for anonymous users when page tools is enabled.

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

I might be wrong but I'm not seeing the pinning functionality in the main menu, only in page tools. It's late in my day so I'll check again tomorrow. Maybe I've got my local setup in a muddle.

Change 897998 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Enable the pinning for anonymous users when page tools is enabled.

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

Change 898782 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@master] Enable pinning for anon main menu whenn page tools is enabled

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

I might be wrong but I'm not seeing the pinning functionality in the main menu, only in page tools. It's late in my day so I'll check again tomorrow. Maybe I've got my local setup in a muddle.

Oh, I thought this ticket was only enabling pinning for the page tools menu. If the main menu is also needed, 898782 should take care of that

Change 898782 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Enable pinning for anon main menu when page tools is enabled

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

nray added a subscriber: Edtadros.

@Edtadros I will add QA steps, sorry for not including them earlier

nray updated the task description. (Show Details)

Test Result - Beta

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

Test Artifact(s):

QA Steps

Visit https://en.wikipedia.beta.wmflabs.org/wiki/Dog?vectorpagetools=1
Open page tools menu in the page toolbar.
✅ AC1: Verify that the menu has the "move to sidebar" option and that it works.

Screen Recording 2023-03-15 at 5.56.33 PM.mov.gif (932×1 px, 736 KB)

Open the main menu.
✅ AC2: Verify that the main menu has the "move to sidebar" option and that it works.

Screen Recording 2023-03-15 at 5.57.00 PM.mov.gif (932×1 px, 696 KB)

Change 902150 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@wmf/1.40.0-wmf.27] Enable pinning for anon main menu when page tools is enabled

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

Change 902150 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.40.0-wmf.27] Enable pinning for anon main menu when page tools is enabled

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

Mentioned in SAL (#wikimedia-operations) [2023-03-22T20:24:51Z] <samtar@deploy2002> Started scap: Backport for [[gerrit:902150|Enable pinning for anon main menu when page tools is enabled (T331657)]]

Mentioned in SAL (#wikimedia-operations) [2023-03-22T20:28:29Z] <samtar@deploy2002> samtar and nray: Backport for [[gerrit:902150|Enable pinning for anon main menu when page tools is enabled (T331657)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2001.codfw.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-03-22T20:36:39Z] <samtar@deploy2002> Finished scap: Backport for [[gerrit:902150|Enable pinning for anon main menu when page tools is enabled (T331657)]] (duration: 11m 47s)

Edtadros removed Edtadros as the assignee of this task.

Test Result - Prod

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

Test Artifact(s):

QA Steps

Visit https://en.wikipedia.org/wiki/Dog?vectorpagetools=1
Open page tools menu in the page toolbar.
✅ AC1: Verify that the menu has the "move to sidebar" option and that it works.

Screen Recording 2023-03-25 at 7.29.03 PM.mov.gif (1×1 px, 822 KB)

Open the main menu.
✅ AC2: Verify that the main menu has the "move to sidebar" option and that it works.

Screen Recording 2023-03-25 at 7.29.57 PM.mov.gif (1×1 px, 647 KB)