Page MenuHomePhabricator

Add icon for "Create account" (when it's in the dropdown menu)
Closed, ResolvedPublic2 Estimated Story Points

Description

Description

When the browser window is below 720px wide, the Create account button moves to the dropdown menu. There is currently no icon displayed next to it:

Screen Shot 2022-04-13 at 1.04.07 PM.png (860×1 px, 212 KB)

To-do

Show the userAdd icon from OOUI next to Create account only when it's in the dropdown menu (i.e. not for cases where the browser window is wider than 720px and Create account is shown in the site header).

image.png (860×1 px, 323 KB)

Event Timeline

ovasileva triaged this task as Medium priority.Apr 13 2022, 5:43 PM

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

[mediawiki/skins/Vector@master] Fix missing create account link icon and ensure icon attributes aren't showing up in rendered HTML

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

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

[mediawiki/core@master] Update create account icon to userAdd

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

Change 780887 abandoned by Bernard Wang:

[mediawiki/core@master] Update create account icon to userAdd

Reason:

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

Change 780887 restored by Bernard Wang:

[mediawiki/core@master] Update create account icon to userAdd

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

Change 780887 merged by jenkins-bot:

[mediawiki/core@master] Update create account icon to userAdd

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

As part of testing I ran the UI regression suite which flagged this change as expected:

Screen Shot 2022-04-14 at 12.58.52 PM.png (1×2 px, 318 KB)

Change 780865 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Fix missing create account link icon and ensure icon attributes aren't showing up in rendered HTML

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

Was there another change related to this that landed recently? For some reason, the latest visual regression test reports are showing it missing again and Beta also shows the icon missing:

Screen Shot 2022-04-15 at 11.11.17 AM.png (576×1 px, 91 KB)

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

[mediawiki/skins/Vector@master] Fix missing create account link follow up

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

Change 782347 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Fix missing create account link follow up

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

@nray i'm seeing it in beta...going to move to signoff but feel free to bring it back if you still have concerns

Screen Shot 2022-04-18 at 11.32.29 AM.png (619×703 px, 77 KB)

Confirming in production on hewiki:

Screen Shot 2022-04-21 at 12.50.27 PM.png (510×484 px, 105 KB)