Page MenuHomePhabricator

IP masking: Make temporary account status and create account actions more prominent via a separate temp account status bar
Closed, ResolvedPublic8 Estimated Story PointsFeature

Description

Background

Following usability testing (see T328368) and design review, there is a proposed revision to clearly delineate the temp account status via a new status "bar" that will sit above the page content.

Proposed design
  • Create a status bar at the top of the page content clearly indicating the temp account status, with clear call-to-actions to Create account or Log in.
  • Same for Desktop and Mobile
  • Providing the Create account and Log in on this bar mitigates the need for adding it to the personal tools links area.

Note: The visual styling is subject to change.

Desktop
image.png (1×2 px, 1 MB)
Mobile
image.png (4×768 px, 1 MB)
Temp account info pop-up - content TBC.
image.png (422×780 px, 54 KB)

Link to WIP figma

Related Objects

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
KStoller-WMF added subscribers: RHo, KStoller-WMF.

I've moved this to the Growth parent epic T326877 so we will be sure to consider this when we focus on that work.
@RHo - just FYI since you are working on initial designs.

RHo renamed this task from IP masking: Should account creation be more prominent for temporary users? to IP masking: Make temporary account status and create account actions more prominent via a separate temp account status bar.Apr 28 2023, 12:11 AM
RHo updated the task description. (Show Details)
RHo updated the task description. (Show Details)

@ovasileva - hope it is fine that I've updated this ticket to reflect the updated designs for the temp account create account and Log in CTA to be placed in a 'temp account status bar' for the Web team. Please revise and separate as you see fit.

ovasileva triaged this task as Medium priority.Apr 28 2023, 2:10 PM

@ovasileva - hope it is fine that I've updated this ticket to reflect the updated designs for the temp account create account and Log in CTA to be placed in a 'temp account status bar' for the Web team. Please revise and separate as you see fit.

This works, thanks @RHo!

@RHo, @Prtksxna - curious what the "i" icon is here - what would it link to?

LGoto set the point value for this task to 8.May 3 2023, 5:24 PM

This is the first time we've had a component that we want to share on mobile and desktop, so I suspect we will want to make this change in core and think carefully through how this is shared. We don't want to reimplement the same styles/template in both Minerva and Vector skin.

Another question @RHo, @Prtksxna - what should the temporary username link link to?

ovasileva raised the priority of this task from Medium to High.May 3 2023, 5:42 PM
RHo updated the task description. (Show Details)

@RHo, @Prtksxna - curious what the "i" icon is here - what would it link to?

Hiya, it would be use the standard info pop-up to show more info about temp accounts. Something like the following but content is all TBC:

image.png (422×780 px, 54 KB)

Another question @RHo, @Prtksxna - what should the temporary username link link to?

Oh good question. I had imagined it would link to their User page (since temp accounts in MVP won't have newcomer homepage), but maybe it should just not be a link at all as it is IP editors to make it less accessible for a temporary person to create a user page. One for @Prtksxna to weigh in on.

@RHo, @Prtksxna - curious what the "i" icon is here - what would it link to?

Hiya, it would be use the standard info pop-up to show more info about temp accounts. Something like the following but content is all TBC:

image.png (422×780 px, 54 KB)

Does this popup already exist or is the idea that web team will create this? If the former, there's a potential performance issue here (that we recently ran into with Vector 2022) so could you subscribe me to the ticket / patch where this got added? thanks in advance!

@RHo, @Prtksxna - curious what the "i" icon is here - what would it link to?

Hiya, it would be use the standard info pop-up to show more info about temp accounts. Something like the following but content is all TBC:

image.png (422×780 px, 54 KB)

Does this popup already exist or is the idea that web team will create this? If the former, there's a potential performance issue here (that we recently ran into with Vector 2022) so could you subscribe me to the ticket / patch where this got added? thanks in advance!

No this is new and is a pop-up that appears after temp account creation and after each subsequent edit with a temp account, so ownership-wise I guess it could be Editing, but maybe it makes sense to be a subtask of this status bar work for Web team if y'all are working on it first?
Interesting to learn more about what the performance issue is?
Depending on severity and effort to use the same pop-up, we could instead change this info icon on the status bar to simply be a "Learn more" link that goes to a general info page about temp accounts.

Interesting to learn more about what the performance issue is?

We ran into some issues relating to this as we've started loading this widget on page load (T334366) and the widget is provided by OOUI and we want to keep the two designs consistent. So might make sense to talk about this component more from a holistic point of view. Perhaps we need to get it into Codex for example.

Jdlrobson changed the subtype of this task from "Task" to "Bug Report".Jun 8 2023, 5:00 PM
Jdlrobson changed the subtype of this task from "Bug Report" to "Feature Request".

Change 928889 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/core@master] [WIP] Basic scaffolding of TempUserBanner skin component

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

Change 929704 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/core@master] As `is-temp` to skin data

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

Change 929705 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/core@master] Move getReturnToParam() to Skin & expose in SkinComponentRegistryContext

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

Change 929715 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/core@master] Create new TempUserBanner skin component

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

Change 929704 abandoned by Jdrewniak:

[mediawiki/core@master] As `is-temp` to skin data

Reason:

probably not necessary (for now).

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

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

[mediawiki/skins/MinervaNeue@master] Enable interface-temp-user-banner styles on Minerva skin

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

Change 929705 merged by jenkins-bot:

[mediawiki/core@master] Skin: Move methods to SkinComponentUtils

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

Change 930849 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Enable interface-temp-user-banner on Vector 2022

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

Change 929715 merged by jenkins-bot:

[mediawiki/core@master] Create new TempUserBanner skin component

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

Change 930849 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Enable interface-temp-user-banner on Vector 2022

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

This one is going to skip QA for now. That will be performed in T339380
I've opened T339379 as follow up work.

Change 930717 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Enable interface-temp-user-banner styles on Minerva skin

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

Change 932531 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] skin: Let Skin::makeSpecialUrlSubpage call correct replacement

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

Change 932531 merged by jenkins-bot:

[mediawiki/core@master] skin: Let Skin::makeSpecialUrlSubpage call correct replacement

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

Change 934418 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Remove "interface-temp-user-banner" option

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

Change 928889 abandoned by Jdrewniak:

[mediawiki/core@master] [WIP] Basic scaffolding of TempUserBanner skin component

Reason:

Abandoned in favour of https://gerrit.wikimedia.org/r/c/mediawiki/core/+/929715/

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