Change 884895 had a related patch set uploaded (by Cyndywikime; author: Cyndywikime):

[mediawiki/core@master] Add a colored background box around a temporary account name

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

I have a couple of questions for @RHo or @Niharika:

• The current patch appears to be working in all desktop skins, but not Minerva. Should it look the same in Minerva, or does that decision belong to the Web team?
• In Figma, the background colour is #eaecf0. There are a couple of different .less variables with this value - which one should we use?

Thanks!

In T325768#8582368, @Tchanders wrote:

I have a couple of questions for @RHo or @Niharika:

• The current patch appears to be working in all desktop skins, but not Minerva. Should it look the same in Minerva, or does that decision belong to the Web team?

The styling should ideally be more consistent everywhere, which is using the same style, but it's trickier in this case because sometimes Minerva (or different skins) styles usernames very differently in general, but sometimes using an avatar or 'incognito" icon to show the difference, sometimes not:

Example	Vector 2022	Minerva
Icon to indicate Anon vs Registered (History page)
Recent Changes - name only

It feels like maybe the decision could be Web team to update the name + icon styling, whether to adjust the icon and/or background (or some other style).

In addition, I wanted to check whether it will update the background colour everywhere with temp username, or only in pages where it is a link? See for example the page to create a Userpage for an IP editor at the moment, there are three differently styled IP addresses here

Vector 2022	Minerva

• In Figma, the background colour is #eaecf0. There are a couple of different .less variables with this value - which one should we use?

Ahh I am not so sure on the less values. The colour in the DSG is @colorGray200, but perhaps you can also see background-color-interactive if the Codex colour tokens are available? This page may or may not be helpful to find the right value: https://doc.wikimedia.org/codex/main/design-tokens/color.html

I think there's an outstanding question from @RHo in this task.

In addition, I wanted to check whether it will update the background colour everywhere with temp username, or only in pages where it is a link? See for example the page to create a Userpage for an IP editor at the moment, there are three differently styled IP addresses here

Vector 2022	Minerva

Second, did we get clarity on the background color?

In T325768#8635290, @Niharika wrote:

I think there's an outstanding question from @RHo in this task.

In addition, I wanted to check whether it will update the background colour everywhere with temp username, or only in pages where it is a link? See for example the page to create a Userpage for an IP editor at the moment, there are three differently styled IP addresses here

Vector 2022	Minerva

In terms of what the current patch does, it only updates the background where it's a link.

@Niharika What do you think should happen in those other places? A related question that has come up in code review - do we want this background to be added on potentially any page where a user link might be shown, or can we limit it to certain pages we know about, e.g. pages that have paginated lists of results?

See @Jdlrobson's comment: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/884895/

Second, did we get clarity on the background color?

It looks as though there's still discussion about where this should go, so we can see what variables are available once that's decided.

In T325768#8668691, @Tchanders wrote:

In T325768#8635290, @Niharika wrote:

I think there's an outstanding question from @RHo in this task.

In addition, I wanted to check whether it will update the background colour everywhere with temp username, or only in pages where it is a link? See for example the page to create a Userpage for an IP editor at the moment, there are three differently styled IP addresses here

Vector 2022	Minerva

In terms of what the current patch does, it only updates the background where it's a link.
@Niharika What do you think should happen in those other places? A related question that has come up in code review - do we want this background to be added on potentially any page where a user link might be shown, or can we limit it to certain pages we know about, e.g. pages that have paginated lists of results?
See @Jdlrobson's comment: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/884895/

In discussion with @Tchanders about this item, my recommendation would be that it could start off with only styling when it's a link, since the reason for the background is to help patrollers and admins spot temp users in places where like RC and Edit history where it can appear in a big list, whereas the example like above being the user page it would be pretty obviously a temp account based on name format.

Second, did we get clarity on the background color?

It looks as though there's still discussion about where this should go, so we can see what variables are available once that's decided.

Thanks @RHo.

@Niharika What do you think should happen in those other places? A related question that has come up in code review - do we want this background to be added on potentially any page where a user link might be shown, or can we limit it to certain pages we know about, e.g. pages that have paginated lists of results?

See @Jdlrobson's comment: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/884895/

@Niharika A related question would be whether we want the background on pages like Special: Permalink (T331008), page information (T331017) and credits (T331021)

In T325768#8672804, @Tchanders wrote:

Thanks @RHo.

@Niharika What do you think should happen in those other places? A related question that has come up in code review - do we want this background to be added on potentially any page where a user link might be shown, or can we limit it to certain pages we know about, e.g. pages that have paginated lists of results?

See @Jdlrobson's comment: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/884895/

@Niharika A related question would be whether we want the background on pages like Special: Permalink (T331008), page information (T331017) and credits (T331021)

Yes, I think it makes sense to add the background on temp usernames on those pages.

Discussed with @Niharika, and the aim is to have the grey background everywhere that a temporary user account link appears, including content pages (article, talk and others).

@Tchanders so would this also apply to the top right of the page UI too? (just confirming)

In T325768#8705514, @Jdlrobson wrote:

@Tchanders so would this also apply to the top right of the page UI too? (just confirming)

@Niharika What do you think?

In T325768#8705514, @Jdlrobson wrote:

@Tchanders so would this also apply to the top right of the page UI too? (just confirming)

@Jdlrobson I can't see the screenshot (restricted access). Do you mean the personal bar? If so, @Prtksxna had some ideas about this.

The personal bar would only be seen by the temp user themselves and so it doesn't need to have a background. That is what we have in the mocks so far.

Thanks @Prtksxna! Could you confirm that we'd like this style everywhere apart from the personal bar? Just to help us understand where to load these styles

We'll do gray everywhere, but there will be exceptions like the personal bar. So, as we had discussed, lets use this task to put it everywhere and I'll file separate tasks for exceptions.

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

[mediawiki/skins/MinervaNeue@master] Enable core interface styles on mobile

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

Change 902186 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Enable core interface styles on mobile

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

Change 884895 merged by jenkins-bot:

[mediawiki/core@master] Add a colored background box around a temporary account name

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

@Cyndymediawiksim Please see the results below, thanks!

OS: macOS 13.2
Browsers: Safari 16.3, Chrome 111, Firefox 111
Environment: Local

Minerva Skin- In Recent Changes & Watchlist, grey spacing is off

Reverted- Wehn reverted did you want the temp user account to be still highlighted in the description? Also once you make a comparison from reverted, it's no longer highlighted in grey.

Personal Bar- Just showing that besides the one above, temp user account is highlighted except for the personal bar. This is as designed though.

In T325768#8736004, @GMikesell-WMF wrote:

@Cyndymediawiksim Please see the results below, thanks!

OS: macOS 13.2
Browsers: Safari 16.3, Chrome 111, Firefox 111
Environment: Local

Minerva Skin- In Recent Changes & Watchlist, grey spacing is off

Reverted- Wehn reverted did you want the temp user account to be still highlighted in the description? Also once you make a comparison from reverted, it's no longer highlighted in grey.

Personal Bar- Just showing that besides the one above, temp user account is highlighted except for the personal bar. This is as designed though.

@GMikesell-WMF , nice catch! Thanks! @Tchanders , thoughts on how we should go about a temporary user link on a reverted edit?

In T325768#8736004, @GMikesell-WMF wrote:

@Cyndymediawiksim Please see the results below, thanks!

OS: macOS 13.2
Browsers: Safari 16.3, Chrome 111, Firefox 111
Environment: Local

Minerva Skin- In Recent Changes & Watchlist, grey spacing is off

Reverted- Wehn reverted did you want the temp user account to be still highlighted in the description? Also once you make a comparison from reverted, it's no longer highlighted in grey.

Personal Bar- Just showing that besides the one above, temp user account is highlighted except for the personal bar. This is as designed though.

@GMikesell-WMF , nice catch! Thanks! @Tchanders , thoughts on how we should go about a temporary user link on a reverted edit?

@Tchanders , thoughts on how we should go about a temporary user link on a reverted edit?

This would be tricky since the link comes from a row in the comment table, rather than being generated as a user link by Linker::userLink (which adds the data we need to display the correct IP address). Hopefully it's not needed since there would likely be other references to this user on the history page, and at worst an admin/patroller could click through to Special:Contributions and find the IP addresses there. Tagging @Niharika and @Prtksxna for visibility.

Minerva Skin- In Recent Changes & Watchlist, grey spacing is off

@GMikesell-WMF Would it be possible to raise a separate ticket for this? We have a few things to fix in Minerva.

In T325768#8738762, @Tchanders wrote:

@Tchanders , thoughts on how we should go about a temporary user link on a reverted edit?

This would be tricky since the link comes from a row in the comment table, rather than being generated as a user link by Linker::userLink (which adds the data we need to display the correct IP address). Hopefully it's not needed since there would likely be other references to this user on the history page, and at worst an admin/patroller could click through to Special:Contributions and find the IP addresses there. Tagging @Niharika and @Prtksxna for visibility.

Ok sounds good. We'll wait to see what they say.

Minerva Skin- In Recent Changes & Watchlist, grey spacing is off

@GMikesell-WMF Would it be possible to raise a separate ticket for this? We have a few things to fix in Minerva.

I created T333474: IP Masking: Minerva Skin- Grey spacing is off in Recent Changes & Watchlist

In T325768#8738762, @Tchanders wrote:

@Tchanders , thoughts on how we should go about a temporary user link on a reverted edit?

This would be tricky since the link comes from a row in the comment table, rather than being generated as a user link by Linker::userLink (which adds the data we need to display the correct IP address). Hopefully it's not needed since there would likely be other references to this user on the history page, and at worst an admin/patroller could click through to Special:Contributions and find the IP addresses there. Tagging @Niharika and @Prtksxna for visibility.

I think it's fine if we don't highlight usernames in the revert comments. @Prtksxna thoughts?

In T325768#8740499, @Niharika wrote:

In T325768#8738762, @Tchanders wrote:

…since there would likely be other references to this user on the history page, and at worst an admin/patroller could click through to Special:Contributions and find the IP addresses there…

I think it's fine if we don't highlight usernames in the revert comments. @Prtksxna thoughts?

Agreed.

Stalling this while we fix T333483: Less_Exception_Parser: File `resources/lib/codex-design-tokens/theme-wikimedia-ui-legacy.less` not found. in interface.less

In T325768#8742844, @Tchanders wrote:

Stalling this while we fix T333483: Less_Exception_Parser: File `resources/lib/codex-design-tokens/theme-wikimedia-ui-legacy.less` not found. in interface.less

This was fixed, and the styles were moved to be with the other Linker::userLink styles. The background colour will be applied less widely now, but consistently with other user link styles. Let's file follow-up tasks if we think these styles are important to add elsewhere.

Here's a separate task for updating Minerva's user link styles: T334581

I find the background color annoying. The recent changes page is already complex with a lot of information, and the blobs with a different background color make the appearance even messier. There's no need to highlight the temporary accounts in particular. Before, there were usernames and IP addresses (numeric). Now there are usernames and the temporary accounts (also largely numeric). Thus, the change has not made the page harder to read and the background color is an unnecessary distraction. Please consider removing it again.

PS. My interest in this is that I am a user on dawiki, which introduced temporary accounts a few days ago, and among other roles is also a patroller on the wiki.

In T325768#10300589, @Dipsacus_fullonum wrote:

I find the background color annoying. The recent changes page is already complex with a lot of information, and the blobs with a different background color make the appearance even messier. There's no need to highlight the temporary accounts in particular. Before, there were usernames and IP addresses (numeric). Now there are usernames and the temporary accounts (also largely numeric). Thus, the change has not made the page harder to read and the background color is an unnecessary distraction. Please consider removing it again.

cc @KColeman-WMF @Niharika