Page MenuHomePhabricator

[Regression, Production] user contributions icon missing on AMC mode
Closed, ResolvedPublic

Description

userContributions menu icon vanished in the AMC menu overflow, during the rename in rSMIN7b4b65a30f05: Reduce 'userContributions' icon code (deployed Mon 3rd). It's in production. We might have caught this sooner but mediawiki.org does not have AMC enabled.

Screen Shot 2020-02-06 at 6.17.16 AM.png (362×445 px, 23 KB)

It's using the class mw-ui-icon-minerva-contributions rather than mw-ui-icon-minerva-userContributions. Might as well upgrade the icon while fixing this to remove it from the repository.

Acceptance criteria

  • Restore the icon

QA steps

With AMC enabled and logged in click the person silhouette icon and verify all the menu items have logos

QA Results

ACStatusDetails
1T244436#5893741
2T244436#5893741

Event Timeline

I am seeing this icon already in minerva mobile (in both modes). Where do we want add it else? Related issue: I notice that, the contribs menu icon vanished in the AMC menu overflow, I am not sure when is started to happen. Is this something known? or is it related to this task?

Screen Shot 2020-02-06 at 6.17.16 AM.png (362×445 px, 23 KB)

Jdlrobson renamed this task from Use 'userContributions' icon from OOUI to [Regression, Production] user contributions icon missing on AMC mode.Feb 6 2020, 8:45 AM
Jdlrobson updated the task description. (Show Details)

Change 570690 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/skins/MinervaNeue@master] Revert "Reduce 'userContributions' icon code"

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

☝️ The above could be deployed during the morning or evening SWAT deployment windows.

@Jdlrobson We just have 'userContributions' in OOUI, several other of the same icons are missing still. From looking at the skin.json I'm not fully clear how the icons are currently put into the modules. 'wikimedia.ui' vs 'skins.minerva.icons.wikimedia' for example. We would have to further separate if we want to use OOUI's 'userContributions'.

Change 570769 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Update wrong icon reference in AMC overflow menu

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

Change 570690 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Revert "Reduce 'userContributions' icon code"

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

Change 570779 had a related patch set uploaded (by Jdlrobson; owner: Phuedx):
[mediawiki/skins/MinervaNeue@wmf/1.35.0-wmf.18] Revert "Reduce 'userContributions' icon code"

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

https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/MinervaNeue/+/570779/ needs to be SWATTed otherwise it will be broken until next Thursday.
There are no SWAT windows on Friday. Possibly @phuedx may be able to make an exception.

The other issues relating to OOUI icons in this ticket are not time sensitive.

Change 570882 had a related patch set uploaded (by Ammarpad; owner: Ammarpad):
[mediawiki/skins/MinervaNeue@master] Use OOUI icon for user contributions menu item

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

Change 570779 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@wmf/1.35.0-wmf.18] Revert "Reduce 'userContributions' icon code"

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

Edtadros added a subscriber: Edtadros.

@phuedx should this be tested on mediawiki.org? If there is a beta environment let me know.

@phuedx Per our conversation I'm putting this in needs more work. As discussed please add any additional comments regarding the state of the task and any testing instructions.

Jdlrobson removed a project: Patch-For-Review.

I recommended that @Edtadros move this to Needs More Work as:

Those patches are associated with T244444 which appears to have been broken out and is in upcoming. Adding this to QA.

Change 570882 abandoned by Ammarpad:
Use OOUI icon for user contributions menu item

Reason:
Done in another patch I0f929f2

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

Test Result

Status: ✅ PASS
OS: macOS Catalina
Browser: Chrome
Device: MBP
Emulated Device: iPhoneX

Test Artifact(s):

QA Steps

With AMC enabled and logged in click the person silhouette icon and verify all the menu items have logos

✅ AC1: This is true on https://m.mediawiki.org/

m.mediawiki.org_wiki_MediaWiki(iPhone X).png (2×1 px, 407 KB)

✅ AC2: This is true on https://en.m.wikipedia.org

en.m.wikipedia.org_wiki_Main_Page(iPhone X).png (2×1 px, 865 KB)