Page MenuHomePhabricator

Legacy Vector: Show temporary account username in personal tools menu
Closed, ResolvedPublic

Description

Summary

Temporary account usernames are not displayed in the personal tools / vector-menu-content section when using legacy Vector (and other skins that rely on SkinTemplate::addPersonalPageItem). This is because the method gates on $this->isNamedUser, which excludes temp users.

Acceptance criteria

  • Temporary account username appears in the personal tools menu for Vector
  • The userTemporary icon is used (not userAvatar) for temp user entries

Event Timeline

Change #1243071 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] Skin: Show temporary account username in personal tools

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

Change #1243072 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/skins/Vector@master] legacy: Show userTemporary icon for temp account usernames

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

Personal toolbar looks like this in ?useskin=vector with the above patches:

image.png (564×27 px, 6 KB)

Change #1243178 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/skins/MinervaNeue@master] Skin: Hide userpage for temp users in menus

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

Change #1243178 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Skin: Hide userpage for temp users in menus

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

Change #1243072 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Skin: Adjust personal tools for temporary users

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

Change #1243071 merged by jenkins-bot:

[mediawiki/core@master] Skin: Show temporary account username in personal tools

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

Change #1243072 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Skin: Adjust personal tools for temporary users

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

As a result of this patch, the top-toolbar in V2010 looks like the following when using a temporary account for me (repro'd locally & on a blank Patch Demo):

screenshot.png (1,033×88 px, 17 KB)

I just want to check if this was intentional, as FWICS it differs slightly from the design in T418220#11644381 (which doesn't seem to include the 'Create account' link after the temp-account's username).

edit: screenshot changed to the correct one; wording

Change #1243072 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Skin: Adjust personal tools for temporary users

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

As a result of this patch, the top-toolbar in V2010 looks like the following when using a temporary account for me (repro'd locally & on a blank Patch Demo):

screenshot.png (1,033×88 px, 17 KB)

I just want to check if this was intentional, as FWICS it differs slightly from the design in T418220#11644381 (which doesn't seem to include the 'Create account' link after the temp-account's username).

edit: screenshot changed to the correct one; wording

Yes, it is intentional (cc @KieranMcCann-WMF)

@A_smart_kitten Yes, it was intentional as we decided it would be useful to surface the ‘Create account’ link close to the temp account username.

dom_walden subscribed.

How it looks on enwiki beta:

ta_vector_ltr.png (586×96 px, 12 KB)

I note that all the links match what we have on Vector 2022 except it doesn't have a Login link.

This does not appear to have affected the way it looks for named users

There are some RTL bugs I raised (T419011, T419012).

Should we revert https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/1243072 ?
We don't show temporary accounts in Wikimedia production to legacy Vector skin and don't support using non-default skins for temporary accounts so this code supports third parties and is not being actively tested.

Should we revert https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/1243072 ?
We don't show temporary accounts in Wikimedia production to legacy Vector skin and don't support using non-default skins for temporary accounts so this code supports third parties and is not being actively tested.

I don't see why that should lead to a revert though? Even if it's not being actively tested, it should be fine to leave as-is with the idea that the skin isn't guaranteed to work with temporary accounts?

Should we revert https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/1243072 ?
We don't show temporary accounts in Wikimedia production to legacy Vector skin and don't support using non-default skins for temporary accounts so this code supports third parties and is not being actively tested.

I don't see why that should lead to a revert though? Even if it's not being actively tested, it should be fine to leave as-is with the idea that the skin isn't guaranteed to work with temporary accounts?

I would agree with this, and suggest we leave things as they are.

We often make changes to menus to support Vector 2022 and Minerva. Given the approach taken in the patch (https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/1243072/7/includes/SkinVectorLegacy.php) the order cannot be guaranteed as stable.

I'm fine with keeping as is if we are okay with things like menu items changing ordering, but I don't want to give a false expectation it is "Active WMF Development" <1>and it will always work the way it currently does.

I see this is marked as "Done" but not resolved. Anything left?

I see this is marked as "Done" but not resolved. Anything left?

Don't think there is, resolving this